Hi both @ashb @feng-tao , I have updated the code bases on your earlier inputs:

- Changed the way to specify bucket/key, in order to be consistent with 
existing S3 operators/sensors.
- Add this class to `docs/code.rst`.
- Test cases are updated (there are two cases to test different argument 
combinations).
- Added a note in the comment (which would be documentation later), 
highlighting that the S3 connection used here must be able to access both 
source and destination bucket/key.

CI passed. PTAL.


[ Full content available at: 
https://github.com/apache/incubator-airflow/pull/3823 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to