[GitHub] [airflow] eladkal commented on pull request #22391: Update the redshift sql operator to accept multiple sql statements

[GitBox <gi...@apache.org>]

eladkal commented on pull request #22391:
URL: https://github.com/apache/airflow/pull/22391#issuecomment-1074404405


   > Can't we just supply a list of SQL statements to the existing operator?
   
   Yes but the goal is to support users who pass string composed of multiple query like:
   
   `SELECT 1; SELECT 2;`
   Most DBs knows how to execute it without any problem (MySQL, PostgresSQL etc..) others like Trino, Presto, Snowflake, Redshift can handle only 1 statement per query.
   
   we can always ask users to pass it as `['SELECT 1', 'SELECT 2']` but that is not that nice and also note that you can't use it with templating. So if you want to use `my_queries.sql ` you can't do that with passing list.


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