jsjasonseba commented on PR #49418:
URL: https://github.com/apache/airflow/pull/49418#issuecomment-2815730037

   > Checking the `services/ui/grid.py` we have some 
`get_task_group_children_getter` there, which I think is supposed to do 
something similiar. Maybe there's something to fix there, I didn’t take a deep 
look yet ?
   
   Hi @pierrejeambrun , thanks for pointing this out, I've looked into the 
code, and it seems like the ordering in frontend relies from 
`StructureDataResponse` which are generated from `get_combined_structure`. 
Currently, `get_combined_structure` does not utilize 
`get_task_group_children_getter` which sets the proper sorting based on Airflow 
cfg. So, there is no issue with the `get_task_group_children_getter` function, 
it is just not utilized to generate the structure response. Let me utilize this 
function instead.


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