judoole commented on a change in pull request #8868:
URL: https://github.com/apache/airflow/pull/8868#discussion_r627947428
##########
File path: airflow/providers/google/cloud/operators/bigquery.py
##########
@@ -546,6 +548,11 @@ def __init__(self,
"the gcp_conn_id parameter.", DeprecationWarning, stacklevel=3)
gcp_conn_id = bigquery_conn_id
+ warnings.warn(
Review comment:
Hey there @telac. I also asked on the Slack channel, and the deprecation
is real and `BigQueryInsertJobOperator` is to be preferred in the future. The
gist of it being;
> BigQueryInsertJobOperator exposes the full capabilities of the API and is
easy to maintain.
> The old operator - BigQueryExecuteQueryOperator needs to be updated every
time a new fields in API is added.
I guess personally I will create an in-house `BigQueryExecuteQueryOperator`
equivalent Operator that extends from `BigQueryInsertJobOperator` and sets the
specific parts we are interested in. For the most parts destination, sql,
time_partitioning and write_disposition.
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
