potiuk edited a comment on issue #7421: [AIRFLOW-XXXX] Slightly speed up mypy 
runs.
URL: https://github.com/apache/airflow/pull/7421#issuecomment-586580143
 
 
   Interesting.  I will check it on my Linux :). BTW . Pylint could be - on the 
other hand - sped up by going parallel - I checked it before and it worked much 
faster if you had multiple CPUs. However we cannot make it parallel because 
that introduces randomness in detecting cycles - so again the same story as 
with mypy - we could get rid of all of the problems by removing stuff from 
airflow.__init__.py and getting rid of those "pull whole airflow when you 
import airflow.<whatever>" thing.
   
   I think it is totally unnecessary problem that we created for ourselves by 
choosing 'airflow' as a public package for all imports from outside (and 
internally). 
   
   And I think Airflow 2.0 is the right time to change it since we are going to 
break the compatibility and we can provide automated 'migration assistants' for 
existing code for 1.0 -> 2.0 migration. 
   
   Maybe we should start discussion on that subject in the devlist? WDYT @ashb?

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