blcksrx commented on PR #25541:
URL: https://github.com/apache/airflow/pull/25541#issuecomment-1205650422
@potiuk
In my opinion, it's better to make this operator as an abstract class and
implement it separately [just overriding external location section] on each
cloud provider cause I believe from user experience it's not pleasant if he/she
already created a cloud connection such as AWS and does not use it on this
operator beside of the security breach on the sql part that needs to pass the
credentials as args.
In addition, using `externalStage` needs manual effort on the user side to
first create a stage then use it on the airflow that reduces the flexibility.
P.S: I created this draft PR to let you know this issue and know your
opinion about it
c.c: @Taragolis
--
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]
