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 2022/04/04 15:49:07 UTC

[GitHub] [airflow] bbovenzi opened a new issue, #22731: Fix the order that tasks are displayed in Grid view

bbovenzi opened a new issue, #22731:
URL: https://github.com/apache/airflow/issues/22731

   The order that tasks are displayed in Grid view do not correlate with the order that the tasks would be expected to execute in the DAG.
   We should update the `task_group_to_tree` function in views.py to better approximate the order that tasks would be run: https://github.com/apache/airflow/blob/main/airflow/www/views.py#L232


-- 
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: commits-unsubscribe@airflow.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [airflow] eladkal commented on issue #22731: Fix the order that tasks are displayed in Grid view

Posted by GitBox <gi...@apache.org>.
eladkal commented on issue #22731:
URL: https://github.com/apache/airflow/issues/22731#issuecomment-1088266869

   i wonder if https://github.com/apache/airflow/issues/13044 share same root cause?


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [airflow] ashb closed issue #22731: Fix the order that tasks are displayed in Grid view

Posted by GitBox <gi...@apache.org>.
ashb closed issue #22731: Fix the order that tasks are displayed in Grid view
URL: https://github.com/apache/airflow/issues/22731


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [airflow] ashb commented on issue #22731: Fix the order that tasks are displayed in Grid view

Posted by GitBox <gi...@apache.org>.
ashb commented on issue #22731:
URL: https://github.com/apache/airflow/issues/22731#issuecomment-1087734110

   We've got this function already, so it _might_ be as simple as using that? https://github.com/apache/airflow/blob/3849b4e709acfc9e85496aa2dededb2dae117fc7/airflow/models/dag.py#L1716-L1726


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [airflow] ashb commented on issue #22731: Fix the order that tasks are displayed in Grid view

Posted by GitBox <gi...@apache.org>.
ashb commented on issue #22731:
URL: https://github.com/apache/airflow/issues/22731#issuecomment-1088475707

   @eladkal Nope, that's purely an artefact of the chart/graphing libary we are using on the front end as expanding the collapsed TG doesn't request any new data -- it's already there in the `nodes` JS var


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org