kurtqq commented on issue #5097: [AIRFLOW-4312] - Add template_fields & 
template_ext to BigQueryCheckO…
URL: https://github.com/apache/airflow/pull/5097#issuecomment-483055209
 
 
   Hi @feluelle  thank you.
   table names aren't always constant. Some users still use old partition 
methodology like:
   ```
   orders_20170101
   orders_20170102
   ```
   So in this use case the table isn't temp and also the table doesn't have 
consist name. Each time the query may refer to another date. This can be used 
with DAG execution_date for example.
   
   Another reason I added it is because `BigQueryGetDataOperator `have 
`table_id `as template field. So same reasons it was added there are applied 
here :) 
   I must say I'm confused why table has so many different references names 
(`table`, `table_id`, `source_project_dataset_table  `etc..) in BigQuery 
operators but I guess that someday it will be refactored and standardized. 

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


With regards,
Apache Git Services

Reply via email to