ferruzzi commented on a change in pull request #15727:
URL: https://github.com/apache/airflow/pull/15727#discussion_r628781574
##########
File path: airflow/providers/amazon/aws/operators/emr_modify_cluster.py
##########
@@ -32,6 +32,8 @@ class EmrModifyClusterOperator(BaseOperator):
:type step_concurrency_level: int
:param aws_conn_id: aws connection to uses
:type aws_conn_id: str
+ (Adding `aws_conn_id` param to template_fields so that it can be
overriden using jinja template from Dags
+ This feature can be useful if user wants to update/override the
aws_conn_id for some kind of Dag Isolation etc)
Review comment:
Doesn't appear that aws_conn_id was actually added below.
--
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]
