You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2019/10/29 18:09:39 UTC

[GitHub] [airflow] mik-laj commented on issue #6465: [AIRFLOW-5803] Rename S3Hook to AWSS3Hook and update import paths

mik-laj commented on issue #6465: [AIRFLOW-5803] Rename S3Hook to AWSS3Hook and update import paths
URL: https://github.com/apache/airflow/pull/6465#issuecomment-547557961
 
 
   Why did you add the AWS prefix? This is not in line with GCP operator naming guidelines. We have no general rules, so we can use the GCP if they make sense.  
   
   Maybe you would like to help with creating a document describing the general principles of naming operators. I will gladly add my comments. A spreadsheet with planned changes would also be helpful. I have these for GCP changes.
   
   CC: @BasPH @potiuk 
   
   https://docs.google.com/document/d/1_rTdJSLCt0eyrAylmmgYc3yZr-_h51fVlnvMmWqhCkY/edit?ts=5bb72dfd# 

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services