You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2022/05/10 19:02:30 UTC

[GitHub] [airflow] uranusjr commented on issue #23623: SnowflakeHook.run() raises UnboundLocalError exception if sql argument is empty

uranusjr commented on issue #23623:
URL: https://github.com/apache/airflow/issues/23623#issuecomment-1122758517

   I think the `execution_info = []` line should be outside of the `for sql_statement in sql:` loop. This also seems to expose a potential bug: currently, if there are multiple SQL statements in `sql`, `execution_info` only contains result from the _last_ SQL statement, not all results from all staements. Not sure if that’s intended though.


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org