rawwar commented on issue #39584:
URL: https://github.com/apache/airflow/issues/39584#issuecomment-2401335394

   @potiuk, I completely forgot about this issue and just started working on 
it. But, I feel this is definitely not a bug, nor require mention of whether to 
use Jinja templating specifically for this Operator. PthonOperator has 
template_fields set to `("templates_dict", "op_args", "op_kwargs")`
   
https://github.com/apache/airflow/blob/f1664674d859a262e93fb3110557a1e71138ca8b/airflow/operators/python.py#L193
   
   So, any operator extending PythonOperator expects template_fields to follow 
Jinja syntax rules. In the documentation, we do not explicitly mention the list 
of template fields for any operator. A thought: we just mentioned that 
templating is possible. L ke 
[here](https://airflow.apache.org/docs/apache-airflow/stable/howto/operator/python.html#templating)
   
   Is it worth adding all possible template fields to all Operator 
documentation?


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