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 2019/10/16 20:17:33 UTC

[GitHub] [airflow] TobKed commented on issue #6351: [AIRFLOW-5682] Allow labels in gcs_to_bq operator

TobKed commented on issue #6351: [AIRFLOW-5682] Allow labels in gcs_to_bq operator
URL: https://github.com/apache/airflow/pull/6351#issuecomment-542873791
 
 
   1. @mik-laj is right, it will be good idea to set default label to something like:
   ```
   from airflow.version import version as airflow_version
   _AIRFLOW_VERSION = 'v' + airflow_version.replace('.', '-').replace('+', '-')
   ```
   2. We have plan to refactor BQ hooks and operators.
   Discussion is here: https://github.com/PolideaInternal/airflow/issues/231
   One of the idea is to pass both the `google.cloud.bigquery.job.QueryJobConfig` class as well as dict representation. (see also: https://googleapis.dev/python/bigquery/latest/generated/google.cloud.bigquery.job.QueryJobConfig.html#google.cloud.bigquery.job.QueryJobConfig)

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services