Allow path in dbapi connect

nevinpuri commented 9 months ago

Just as you can pass a path argument to the dbapi.connect() function (enabling persistency). This will allow similar functionality as the Session api already provides, but using the dbapi.

Changelog category (leave one):

Improvement

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):

Added path parameter to dbapi

Documentation entry for user-facing changes

Not sure this requires a documentation entry, but here it is: path: the folder path to use for database data on disk.
[x] Documentation is written (mandatory for new features)

Information about CI checks: https://clickhouse.com/docs/en/development/continuous-integration/

CLAassistant commented 9 months ago

All committers have signed the CLA.

nevinpuri commented 9 months ago

Don't merge yet, I just realized this change isn't fully working

nevinpuri commented 9 months ago

I've updated all the code. Should work but I haven't tested.

auxten commented 8 months ago

Please add tests for your modification. Also, there is a CLA need to sign.

nevinpuri commented 6 months ago

@auxten I have added a test. It all should work now 👍

auxten commented 6 months ago

Fixing some test issue organizations-2000000.zip link expired. Just use organizations-500000.zip

chdb-io / chdb