[GitHub] [airflow] RosterIn commented on a change in pull request #6578: [DEPENDS ON #6575][DEPENDS ON #6577][AIRFLOW-5907] Add S3 to MySql Operator

[GitBox <gi...@apache.org>]

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_r346814710
 
 

 ##########
 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:
   I think you need to specify it's CSV file.
   Other operators supports json as well - this one does not

----------------------------------------------------------------
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