o-nikolas commented on code in PR #22758:
URL: https://github.com/apache/airflow/pull/22758#discussion_r852286408
##########
airflow/providers/amazon/aws/operators/s3.py:
##########
@@ -318,6 +318,94 @@ def execute(self, context: 'Context'):
)
+class S3CreateObjectOperator(BaseOperator):
+ """
+ Creates a new object from a given string or bytes.
Review Comment:
> and we actually discovered during this discussion that the original
intention was for HttpToS3Operator
I think the conversation got a bit muxed by one example being given at the
top then an assumption being made that such an example is the only applicable
usecase ever possible. I have mentioned examples in my previous two comments
about usecases for which the HttpToS3Operator would _not_ help and this
operator would. But, as always, I'm happy to disagree a commit if you deem it
dangerous or reckless to merge this than we can continue to use the hook
directly.
--
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]
