avocadomaster opened a new pull request #16216:
URL: https://github.com/apache/airflow/pull/16216


   
   As far as I understand the `load_file` function in the S3 hook expects key 
before bucket name.
   
   
https://github.com/apache/airflow/blob/d245992d2ac0d781b6b55fd030a076ca5c799bf7/airflow/providers/amazon/aws/hooks/s3.py#L466
   
   The `AwsGlueJobOperator` currently calls the `load_file` function with 
`s3_bucket` before `key`. 
   
   
https://github.com/apache/airflow/blob/d245992d2ac0d781b6b55fd030a076ca5c799bf7/airflow/providers/amazon/aws/operators/glue.py#L104
   
   This PR changes that order to comply with the order of `load_file`
   
   


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


Reply via email to