-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
fix: rendering of more complex pipelines tree view #14595
fix: rendering of more complex pipelines tree view #14595
Conversation
Hi there 👋 Thanks for your contribution! The OpenMetadata team will review the PR shortly! Once it has been labeled as Let us know if you need any help! |
Hi there 👋 Thanks for your contribution! The OpenMetadata team will review the PR shortly! Once it has been labeled as Let us know if you need any help! |
Hi there 👋 Thanks for your contribution! The OpenMetadata team will review the PR shortly! Once it has been labeled as Let us know if you need any help! |
hi @mgorsk1 Thanks for the PR. We will review and merge it in |
@ShaileshParmar11 did your DAG changes for this merged in? can you work with @mgorsk1 to get this PR in. |
Yes my UI changes merged! @harshach |
|
so are we happy about this PR @ShaileshParmar11? I have fixed all the linting issues and ci seems to be satisfied. |
Thank you @mgorsk1 |
🤝🏼 |
Describe your changes:
Fixes 14599
I worked on fixing executionUtils getTreeData method because it's current implementation is not suitable for precise depicting parent > child relationships between pipeline tasks in Tree view of Executions tab.
Before (notice lack of process_task_X in Tree view of Executions tab)
Nagranie.z.ekranu.2024-01-5.o.17.47.23.mov
After:
Nagranie.z.ekranu.2024-01-5.o.18.01.13.mov
Type of change:
Checklist:
Fixes <issue-number>: <short explanation>