ferruzzi edited a comment 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 2020-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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to