ecodina commented on PR #39701: URL: https://github.com/apache/airflow/pull/39701#issuecomment-2118806064
> Good work, just a couple of questions for my understanding: > > 1. What's "All states" required for? Maybe "Running" and "Failed" can just work with the existing "All", "Active", "Paused"? I added this button in order to reset the Running/Failed filter. For example: - I select "Active" - I select "Failed" I now want to see "Active" + "Any state". What should I do? By adding this button, you can do it. Moreover, you clearly show that the properties filtered by each set is different. One filters Dag properties, while the other focuses on DagRun > 2. Regarding the new SQL to count all DAG runs - not sure what the best approach is here. Do we want to see how many unique DAGs are currently running regardless of how many instances of the same DAG are running (current implementation)? Or do we want to see the total number of DAG runs (even if all come from the same DAG, say if you're doing a backloading). > > Perhaps some comments from the Airflow maintainers would help here. While I was developing this I found it very confusing to see a "Running (8)" but only 1 DAG on the list (with 8 running DAG runs). I believe that the # of DAGs running is better displayed on the Cluster Activity page. > > P.S. did you mean to link #39675 instead of #30429 by any chance? You are right! Thanks for seeing this! -- 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]
