josh-fell opened a new pull request #20347:
URL: https://github.com/apache/airflow/pull/20347


   Closes: #20333
   
   Currently there is a datatype check on `source_objects` within the 
`GCSToBigQueryOperator.__init__()` method. This is a template field for the 
operator. If a user tries to pass in an `XCom` value that will be rendered as a 
list (i.e. `render_template_as_native_obj=True` at the DAG level), the datatype 
check incorrectly assumes the value is a string and wraps the value in a list. 
When the `XCom` value is then rendered during `execute()` the value becomes a 
`List[List]` instead of the desired `List` type. 
   
   Moving this datatype check into the `execute()` method of the operator will 
allow any template rendering to occur before the check is performed.
   
   ---
   **^ Add meaningful description above**
   
   Read the **[Pull Request 
Guidelines](https://github.com/apache/airflow/blob/main/CONTRIBUTING.rst#pull-request-guidelines)**
 for more information.
   In case of fundamental code change, Airflow Improvement Proposal 
([AIP](https://cwiki.apache.org/confluence/display/AIRFLOW/Airflow+Improvements+Proposals))
 is needed.
   In case of a new dependency, check compliance with the [ASF 3rd Party 
License Policy](https://www.apache.org/legal/resolved.html#category-x).
   In case of backwards incompatible changes please leave a note in 
[UPDATING.md](https://github.com/apache/airflow/blob/main/UPDATING.md).
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to