You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2020/06/04 01:33:21 UTC

[GitHub] [airflow] potiuk commented on pull request #9132: Split utils sh

potiuk commented on pull request #9132:
URL: https://github.com/apache/airflow/pull/9132#issuecomment-638548194


   Following the discussion with @dimberman (and recent slack conversation with @ashb about `utils` as being bad name) -> I split the _utils.sh into meaningful and much smaller libraries - each responsible for pretty much one thing only.
   
   It should be much easier to understand what's going on and where to search for corresponding part of the bash script. If we ever decide to reimplement some parts of it in a different languages, that might also be good idea to understand what Breeze and the scripts are actually doing.
   


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