kazanzhy commented on issue #9957: URL: https://github.com/apache/airflow/issues/9957#issuecomment-1272106082
@eladkal When we moved to the separate `common.sql` provider it was untyped. After this commit, these types were added https://github.com/apache/airflow/pull/25299/files#diff-6e1b2f961cb951d05d66d2d814ef5f6d8f8bf8f43c40fb5d40e27a031fed8dd7 Oh, I just saw that it's a 2-yeas old issue. There's a simple `execute` method used here, therefore I think we have the wrong type here. At least for Postgres https://www.psycopg.org/docs/cursor.html#execute We could try to use `DbApiHook.run` for the `DbApiHook.get_records` and `DbApiHook.get_first`. I'll try to implement it -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
