[GitHub] [airflow] kazanzhy commented on pull request #20119: Remove cursor methods calls from GCSToBigQueryOperator

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

kazanzhy commented on PR #20119:
URL: https://github.com/apache/airflow/pull/20119#issuecomment-1182595462

   Very good work with the https://github.com/apache/airflow/pull/24363.
   
   As I wrote earlier in this discussion it should be the place with all this routine dict creation.
   I tried to do it in my work, but it's awfully inconvenient. It's much easier to use Pycharm and autocomplete than explore Google docs to understand where would be placed exactly this argument.
   
   My previous idea was to keep it in hook, but probably we could think about the handler.
   Recently I updated my data pipelines and must say that the combination of `ClusterGenerator` and `DataprocCreateClusterOperator` is fantastic.I think it's a good balance between flexibility and code overload.
   I'm working on another PR and it's quite big. Maybe @cswpy has time?


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