galak75 commented on issue #4743: [AIRFLOW-3871] render Operators template 
fields recursively
URL: https://github.com/apache/airflow/pull/4743#issuecomment-472996201
 
 
   @bjoernpollex-sc, thank you for your feedback. 
   
   the statement below works fine:
   ```python
   p = pathlib.Path('/tmp/{{ dag.dag_id }}')
   ```
   
   Then, I have an idea to solve [the limitation you 
described](https://github.com/apache/airflow/pull/4743#issuecomment-472735428): 
why not add an additional `render_template` hook in addition to the 
`template_fields` one. Then one could define a `template_fields` list, while 
another could define a custom `render_template` method (for cases such the one 
you described).
   
   Both templating hooks could be used together to provide even more 
flexibility.
   
   What do you think about it?

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to