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/10/22 22:13:47 UTC

[GitHub] [airflow] dstandish commented on pull request #27204: Fix system tests for SnowflakeOperator

dstandish commented on PR #27204:
URL: https://github.com/apache/airflow/pull/27204#issuecomment-1287931077

   > I created a missing SQL file that was used by system tests. Unfortunately, this file cannot contain the license header, because Snowflake interprets the license header as a new SQL query, and we can only send one SQL query per request. This is something that might be worth fixing, but for now, I'd like to run system tests without making any changes to the operator.
   
   @mik-laj you could consider doing something like
   
   ```python
   '\n'.join(x for x in Path(...).read_text().splitlines() if not x.startswith('#'))
   ```


-- 
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