-
Notifications
You must be signed in to change notification settings - Fork 230
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
Bug when passing a query to execute that is a psycopg2 SQL object #377
Labels
Comments
zschumacher
changed the title
Bugs when passing a query to execute that is a psycopg2 SQL object
Bugswhen passing a query to execute that is a psycopg2 SQL object
Sep 21, 2020
zschumacher
changed the title
Bugswhen passing a query to execute that is a psycopg2 SQL object
Bug when passing a query to execute that is a psycopg2 SQL object
Sep 21, 2020
Can you create a test? |
The issue still persists. |
A test will help to move it. |
spirkaa
added a commit
to spirkaa/gia-api
that referenced
this issue
Dec 7, 2023
workaround for Suor/django-cacheops#377
Hit the same issue while using django-bulk-load. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
When you pass a
SQL
object frompsycopg2.sql
, we run into an unexpected error in theis_sql_dirty
function. When we hit the line that tries to dosql.lower()
, we may have passed in aSQL
object, resulting inpackages/cacheops/transaction.py", line 113, in is_sql_dirty sql = sql.lower() AttributeError: 'Composed' object has no attribute 'lower'
Perhaps it would be reasonable to add a block such as
Would love other thoughts on this.
The text was updated successfully, but these errors were encountered: