gdavoian commented on PR #35546: URL: https://github.com/apache/airflow/pull/35546#issuecomment-1803528433
@utkarsharma2 But the `email` field isn't added to `template_fields` by default (`BaseOperator.template_fields` is empty), it's up to developers of third-party/custom operators to add it to their operators. So if they really want `email` to be templated and are aware of that overhead, then why not? :) Honestly, I don't know how that caching works, but we use templating heavily (not only for email lists but also for things like the links to docker images, etc.), and currently, the only thing we can't really template is the `email` field coming from `BaseOperator` (e.g., we can easily template the `image` field of `KubernetesOperator` etc.). -- 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]
