[GitHub] [airflow] eladkal commented on pull request #23971: Unify DbApiHook.run() method with the methods which override it

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

eladkal commented on PR #23971:
URL: https://github.com/apache/airflow/pull/23971#issuecomment-1221389283

   This looks like a PL/SQL not SQL.
   I dont think it was ever intended to work with SqlOperator of any kind.
   When we talked about the unified solution I didnt consider PG/PL SQL (or equivalent) for the simple reason that its not pure SQL.
   
   I guess if you want it to work with Sql then you should wrap your statments with store procedure and then invoke it by SELECT statment.
   


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