-
Notifications
You must be signed in to change notification settings - Fork 484
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.
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
Metadata and schemas other than public #1155
Comments
As this seems to be backwards compatible (right?), I see no reason not to add support for specifying a different schema — please submit your PR and I'll try to merge it soon. TIA! |
The api does not change, but the sematics of the connection string changes for the two databases I have implemented the change for.
Everything else is unchanged. PR in #1156 |
Yes - I'm working on bringing metadata up to par for ODBC - but it's going a bit slow for the time being. |
Ah, I thought it was already done since the PR is closed. My bad 🤷 |
The metadata code introduced by #480 hardcodes the used schema to public.
This is not great if you want to investigate any other schema than the public schema.
I've created test in my own fork of SOCI - and I'm ready to make a PR with that code - which stores the schema on the session object (default is public) and allows backends to change this based on whatever, but most likely a type of connection string.
E.g. for postgres "user=postgres host=127.0.0.1 port=5432 dbname=postgres options='--search_path=my_schema' password=mypwd" will select my_schema for the metadata instead of forcing public, or for mysql the database name will be used for the schema name - which is also quite sensical.
I have only implemented this in the mysql and postgresql backends as I don't have experience with- or any test DBs for any of the other supported DBs.
The text was updated successfully, but these errors were encountered: