feluelle edited a comment on issue #6578: [AIRFLOW-5907] Add S3 to MySql 
Operator
URL: https://github.com/apache/airflow/pull/6578#issuecomment-593462941
 
 
   @RosterIn WDYT of this now?
   
   I personally don't like the `extra_options` (I added to `mysql` myself to 
`bulk_load_custom` :D). So now you are able to specify these options in the 
transfer operation. So you can load json or csv or whatever as long as LOAD 
DATA supports it.
   
   **EDIT:**
   I don't like the `extra_options` because it is not fully clear what options 
of https://dev.mysql.com/doc/refman/8.0/en/load-data.html these can be.
   But the issue is more because of `bulk_load_custom` not because of the 
implementation of the S3ToMySql, I would say.

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

Reply via email to