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 2021/09/05 09:31:32 UTC

[GitHub] [airflow] potiuk edited a comment on pull request #17329: Split sql statements in DbApi run

potiuk edited a comment on pull request #17329:
URL: https://github.com/apache/airflow/pull/17329#issuecomment-913117409


   One other comment here: It would be great to add unit tests for more complex queries like: https://stackoverflow.com/questions/69049321/javascript-udf-fails-in-airflow-but-works-in-snowflake/69049529#69049529  
   
   There are some DB's (like snowflake) that allow to embed other languages in SQL (like javascript in this case). The split used in Snowflake Operator is not resilient to that when `;` were part of the Javascript. I wonder if sqlparse will handle it properly.


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