-
Notifications
You must be signed in to change notification settings - Fork 13
Problem creating connection to PostgreSQL database #12
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
Comments
Emilio, it seems to me it is not a case-sensitive problem. When looking into ODM2 sqlalchemy models, in case of postgreSQL, there need "odm2" schema. For example, "odm2.Variables". |
Thanks, @cdesyoun! Yes, I use an "odm2" schema name. That's the convention we've adopted across ODM2, so the ODM2PythonAPI needs to handle it. But if a query is being issued that does have a Hopefully Stephanie can look into it on Monday, and with the information she provides we'll be able to fix it quickly. |
This issue has been addressed in the setup branch |
Excellent. I'll test with postgresql. |
Problem fixed (in setup_em, not setup, branch). I no longer get this error during testing with postgresql. Closing issue. |
After installing ODM2PythonAPI using the overhauled setup I described in issue #11 (with the
setup_em
branch I've created), I got an error trying to make a test connection to one of my local PostgreSQL ODM2 databases:The messages
relation "Variables" does not exist
andLINE 2: FROM "Variables"
suggest that it's trying to issue a query using a case-sensitive (double-quoted) table name for "Variables". I was able to track the error to probably this line in odm2api/ODMconnection.py, but I wasn't able to go beyond that.@sreeder, can you look into it? I can help fix it if it's a case issue and you can point me to where the problem is. BTW, this may be related to the case handling issues discussed a few months ago in issue #2. Back then I was able to change the code and get it to work for me with postgresql, but that was before you overhauled the case handling.
The text was updated successfully, but these errors were encountered: