khilawar4 commented on a change in pull request #15727:
URL: https://github.com/apache/airflow/pull/15727#discussion_r629762502
##########
File path: airflow/providers/amazon/aws/operators/s3_bucket.py
##########
@@ -37,12 +37,16 @@ class S3CreateBucketOperator(BaseOperator):
running Airflow in a distributed manner and aws_conn_id is None or
empty, then default boto3 configuration would be used (and must be
maintained on each worker node).
+
Review comment:
Its good idea to give some info as part of the comments so that users
are aware why aws_conn_id was added as part of the template_fields and what
other benefit it could have.
--
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]
