RosterIn commented on a change in pull request #6578: [DEPENDS ON #6575][DEPENDS ON #6577][AIRFLOW-5907] Add S3 to MySql Operator URL: https://github.com/apache/airflow/pull/6578#discussion_r347757745
########## File path: airflow/operators/s3_to_mysql.py ########## @@ -0,0 +1,65 @@ +from airflow.hooks.mysql_hook import MySqlHook +from airflow.models import BaseOperator +from airflow.providers.aws.hooks.s3 import S3Hook +from airflow.utils.decorators import apply_defaults + + +class S3ToMySqlTransfer(BaseOperator): + """ + Loads a file from S3 into a MySQL table. Review comment: You can follow how it works in MySqlToGoogleCloudStorageOperator https://github.com/apache/airflow/blob/master/airflow/operators/mysql_to_gcs.py#L36 It support both Json and csv Personally I really like the idea of BaseSQLToGoogleCloudStorageOperator as I believe it's the same case here but that's your call. ---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services
