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/07/30 00:05:00 UTC

[GitHub] [airflow] josh-fell commented on pull request #25412: Fixing JdbcOperator non-SELECT statement run

josh-fell commented on PR #25412:
URL: https://github.com/apache/airflow/pull/25412#issuecomment-1200042069

   >My main question for other maintainers is which solution is better
   > - adding parameter fetch_results: bool
   > - give users the ability to pass their own handlers
   
   It might not be what is _better_, but perhaps why not do both? Have users control if they want to fetch results from the query they execute and the option to customize how to handle said results (with `fetch_all_handler` as a default even).
   
   


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