uranusjr commented on a change in pull request #20945:
URL: https://github.com/apache/airflow/pull/20945#discussion_r787585718
##########
File path: airflow/models/baseoperator.py
##########
@@ -467,11 +476,10 @@ class derived from this one results in the creation of a
task object,
:type doc_yaml: str
"""
- # For derived classes to define which fields will get jinjaified
- template_fields: Sequence[str] = ()
- # Defines which files extensions to look for in the templated fields
- template_ext: Sequence[str] = ()
- # Template field renderers indicating type of the field, for example sql,
json, bash
+ # Implementing Operator.
+ template_fields: Collection[str] = ()
Review comment:
It’s the other way around, from Sequence to a Collection; Collection is
looser than Sequence and should work in subclasses (I think; we can always
change this back).
--
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]
