apache / apache/arrow-adbc

Python - Autocommit two ways of setting it

Open
#3,853 1 comment 0 reactions 0 assignees View on GitHub
help wanted Type: enhancement
Dominant language
C#
Stars
627
Forks
217
Avg merge
17h
Merged PRs (30d)
57

Description

### What would you like help with?

In python's adbc-driver-manager there are two ways of setting `autocommit` in the `connect` method one is through the `conn_kwargs` the other is through `autocommit`.
the first uses `set`Option _pre-init_ and the second uses a _post-init_ `setOption`.

in general, i think a pre-init is preferable and the connect is able to use it. My question is why not?

context: it doesn't hurt functionality but because `autocommit` can be set in two ways at the same time can sometimes cause unexpected behavior / be confusing.

Contributor guide

Open the contributing guide

Research direction

Start in the adbc-driver-manager Python connect method and compare the conn_kwargs pre-init setOption path with the autocommit post-init setOption path. Trace how both options are applied and document or test the resulting behavior; done requires a clear decision about whether the duplicate setting paths should remain.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
databases
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.