Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add option to hide Datasets in Graph View #41003

Closed
1 of 2 tasks
ttzhou opened this issue Jul 24, 2024 · 4 comments · Fixed by #41200
Closed
1 of 2 tasks

Add option to hide Datasets in Graph View #41003

ttzhou opened this issue Jul 24, 2024 · 4 comments · Fixed by #41200
Assignees
Labels
area:UI Related to UI/UX. For Frontend Developers. kind:feature Feature Requests

Comments

@ttzhou
Copy link
Contributor

ttzhou commented Jul 24, 2024

Description

If we have a lot of datasets generated by tasks in a DAG, it can clutter up the Graph view, where sometimes we just want to see the relations between the actual tasks.

Is it possible to add an option (either some kind of toggle in webserver UI and/or an environment variable) that can allow us to hide all the datasets in the Graph view?

Use case/motivation

The hope is that we allow users the ability to reduce clutter in their Graph view when trying to debug/trace lineage of tasks, when many dataset blocks are present.

Related issues

I don't think so, but would be very happy to close this if there already is one!

Are you willing to submit a PR?

  • Yes I am willing to submit a PR!

Code of Conduct

@ttzhou ttzhou added kind:feature Feature Requests needs-triage label for new issues that we didn't triage yet labels Jul 24, 2024
Copy link

boring-cyborg bot commented Jul 24, 2024

Thanks for opening your first issue here! Be sure to follow the issue template! If you are willing to raise PR to address this issue please do so, no need to wait for approval.

@ttzhou
Copy link
Contributor Author

ttzhou commented Jul 24, 2024

I may take a stab at a PR when I have some time, but wanting to track my thoughts with this issue.

@dosubot dosubot bot added the area:UI Related to UI/UX. For Frontend Developers. label Jul 24, 2024
@shahar1 shahar1 removed the needs-triage label for new issues that we didn't triage yet label Jul 26, 2024
@uranusjr
Copy link
Member

I think we should also have an option to hide the DAGs too (the other way around). cc @bbovenzi sicne I think there’s some planned improvements on the view for 3.0.

@uranusjr uranusjr added this to the Airflow 3.0.0 milestone Jul 31, 2024
@bbovenzi
Copy link
Contributor

bbovenzi commented Jul 31, 2024

Yeah, both graphs can have a lot more parameters. Some thoughts:

DAG graph view:

  • add sensors and triggers as external dependencies like we already have for datasets
  • have the backend pass dataset conditions and dataset aliases as nodes and edges without the UI having to calculate it
  • include a show_external_deps param, when it is true the graph will show datasets/aliases/sensors/triggers. Then the UI can also include a button to toggle datasets on/off

Datasets Dependency Graph

  • filter_by param to pass a dag_id, dataset alias name, or a dataset uri to only show the graph connected in some way to that graph
  • only_datasets to only render datasets and aliases to see how data flows without worrying about what dags are involved
  • add dataset conditions nodes like the DAG graph now has
  • add a new endpoint with an event (dag_id + run_id or uri + timestamp) and it will return all of the events upstream and downstream of that event.

@kaxil kaxil added the airflow3.0:candidate Potential candidates for Airflow 3.0 label Jul 31, 2024
@bbovenzi bbovenzi removed the airflow3.0:candidate Potential candidates for Airflow 3.0 label Aug 5, 2024
@bbovenzi bbovenzi modified the milestones: Airflow 3.0.0, Airflow 2.10.0 Aug 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:UI Related to UI/UX. For Frontend Developers. kind:feature Feature Requests
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants