FanatoniQ commented on PR #25430: URL: https://github.com/apache/airflow/pull/25430#issuecomment-1202101011
Pep and sqlalchemy states `rowcount` may be missleading and is only usable with UPDATE DELETE... Regarding `rownumber` it's listed in optional dbapi2 extensions... I don't think that the fact that we are using the same cursor in the run for loop causes an issue. Sqlalchemy's implementation of `returns_rows` is simply like they say on the documentation : `cursor.description is not None`. -- 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]
