MarosMK commented on PR #30284:
URL: https://github.com/apache/airflow/pull/30284#issuecomment-1488935535

   @uranusjr
   The code changes were updated (no caching of group_id) so hopefully there is 
now less risk at stake and the speedup is the same as before. 
   For the time improvement, if we take nested groups of 3 (each task has 3 
child tasks) and we go into 'depth' of 6 layers, the dag loading is faster by 
7.7s. Or 4 nested groups with 5 layers, it is faster by 3.5s. It depends on how 
many layers you have and how many task in each layer. 
   We do use a lot of nested groups so this would be an improvement. 


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