potiuk commented on code in PR #34369: URL: https://github.com/apache/airflow/pull/34369#discussion_r1327184560
########## airflow/providers/google/cloud/transfers/gcs_to_samba.py: ########## Review Comment: Also one more thought that this discussion inspired (and maybe you should consider that as an option in your proposal). If you want to also optimise for findability - why don't we simply import the Google transfer operator In the samba provider in transfer? Just import It there In similarly named package. (And same for all other transfers) This seems to solve the problem you mentioned -of finding the operator - user will FIND and will be able to use it in both providers. They will be able to find them in both places in docs and import from whatever provider they choose to. It will work. That will remove the user out of the discussion because then it won't matter for the user. It will leave only the maintainers in the discussion and there I believe the currently agrees definition is good. -- 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]
