ashb commented on a change in pull request #20945:
URL: https://github.com/apache/airflow/pull/20945#discussion_r787571644
##########
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:
Why change from a collection to a sequence? (This might cause knock-on
impacts for typing of all other operators)
--
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]
