[GitHub] [airflow] uranusjr commented on a diff in pull request #28705: Update S3ToRedshift Operator docs to indicate multiple key functionality

[GitBox <gi...@apache.org>]

uranusjr commented on code in PR #28705:
URL: https://github.com/apache/airflow/pull/28705#discussion_r1061173156


##########
airflow/providers/amazon/aws/transfers/s3_to_redshift.py:
##########
@@ -42,7 +42,7 @@ class S3ToRedshiftOperator(BaseOperator):
     :param schema: reference to a specific schema in redshift database
     :param table: reference to a specific table in redshift database
     :param s3_bucket: reference to a specific S3 bucket
-    :param s3_key: reference to a specific S3 key
+    :param s3_key: reference either to a specific S3 key or a set of keys or folders sharing that prefix

Review Comment:
   What does “a set of keys” mean? Does it has to be a Python set, or is the term being used more liberally? If the latter case I think _collection_ is a more common term.



-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org