Skip to content

Add the possibility to connect to a specific database #61

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
andrejtonev wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Add the possibility to connect to a specific database #61

andrejtonev wants to merge 3 commits into from

Conversation

@andrejtonev
Copy link

@andrejtonev andrejtonev commented Mar 29, 2024

Add multi-tenant capability.

conn = mgclient.connect(address...., database="db")

will return a connection that is limited to the defined database.
@andrejtonev andrejtonev requested a review from Josipmrden March 29, 2024 15:59
@andrejtonev andrejtonev marked this pull request as ready for review April 5, 2024 08:43
@andrejtonev andrejtonev self-assigned this Apr 5, 2024
@andrejtonev andrejtonev added the Docs needed Docs needed label Apr 5, 2024
@andrejtonev
Copy link
Author

andrejtonev commented Apr 5, 2024

@kgolubic We have added the ability to define the landing database via pymgclient.
mgclient.connect(database="your db", ...)
The connection is then locked to the defined db, you cannot switch to another one later on.
@andrejtonev andrejtonev marked this pull request as draft April 5, 2024 08:53
@andrejtonev andrejtonev marked this pull request as ready for review April 5, 2024 10:08
@andrejtonev
Copy link
Author

andrejtonev commented Apr 5, 2024

FYI @Josipmrden this PR requires a fix in memgraph.
Should be delivered via memgraph/memgraph#1884
@gitbuda gitbuda mentioned this pull request Aug 17, 2024
Closed
7 tasks
@gitbuda gitbuda mentioned this pull request Apr 30, 2025
Open
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Docs needed Docs needed

2 participants

@andrejtonev