[GitHub] [airflow] ferruzzi commented on issue #20139: Restructure Amazon provider operators files

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

ferruzzi commented on issue #20139:
URL: https://github.com/apache/airflow/issues/20139#issuecomment-992932416


   Hey cool initiative, I definitely agree with this plan! Sorry for the delay in response, I was enjoying a seaside cabin without internet last week. 🏖️   I will take a closer look at what's been claimed and add some work on this tomorrow.
   
   For the Glue question,  the docstring on `glue.py:L28` says "Interact with AWS Glue - create job, trigger, crawler"; looking at the history that was never included in the same file.   Maybe that as the intent all along?
   
   glue.py was created 200-05-02 by abdulbasitds
   glue_crawler.py was created 2021-01-24 by Marshall Mamiya
   
   from the initial PR [here](https://github.com/apache/airflow/pull/6007) it looks like there was some copy/pasta from someone else's fork so maybe the plan was to combine them all along?  I don't really use Glue so maybe someone with a better working knowledge of how it's actually used in the wild would have an opinion on this, and I'd definitely defer to that, but it looks to me like maybe we can combine them.


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

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org