I'm having a hard time understanding this logic with all the nested loops and 
type checking.

I think it'd be easier if we started off by setting `configuration = 
copy.deepcopy(api_resource_configs)` just before 
https://github.com/apache/incubator-airflow/blob/e933ee8e52e0449d88bac0c83f6ee06729988343/airflow/contrib/hooks/bigquery_hook.py#L602

We should warn anywhere where we set a `key` by one of the other arguments but 
it was already present in `api_resource_configs`.

[ Full content available at: 
https://github.com/apache/incubator-airflow/pull/3733 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to