neo4j / neo4j/neo4j-python-driver

Consider officially supporting the database option for driver()

Open
#1,322 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

feature request
Dominant language
Python
Stars
1k
Forks
213
Avg merge
1d 3h
Merged PRs (30d)
19

Description

Consider moving https://github.com/neo4j/neo4j-python-driver/blob/e97a122a40912c1016643a0fe8dda97f69bbc4a3/src/neo4j/_sync/driver.py#L164 from # undocumented/unsupported options to documented/supported

The common use-case is to use the same database for the whole app. Using a different one in a specific session is the exception.

This option already makes it possible to instantiate the driver with desired options in one place (like Django's settings.py) and share that object across the project.

Would much appreciate making it officially supported.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start in src/neo4j/_sync/driver.py at line 164 and inspect how the database option is currently handled and described. Determine what is required to move it from undocumented/unsupported to documented and supported; done means the option is officially supported for configuring a shared driver database.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
databases
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
58/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.