-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
NIFI-12485: Lineage Graph #8173
Conversation
- Lineage.
- Adding context menu to lineage graph. - Refactoring canvas context menu to promote reuse.
- Lineage timeline slider.
Will review. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I noticed some rendering issues in the lineage graph sometimes when expanding nodes (in particular FORKs):
It doesn't happen on all events, but does happen consistently with the certain events. One note, when setting a breakpoint in the lineage.component.ts line 900, and stepping through the expand action, the issue does NOT present itself. So it is probably something with in the timing 😞
Also, a few things inline...
...i/src/app/pages/provenance/ui/provenance-event-listing/provenance-event-listing.component.ts
Show resolved
Hide resolved
...eb/nifi-web-frontend/src/main/nifi/src/app/pages/provenance/state/lineage/lineage.effects.ts
Outdated
Show resolved
Hide resolved
...eb/nifi-web-frontend/src/main/nifi/src/app/pages/provenance/state/lineage/lineage.effects.ts
Outdated
Show resolved
Hide resolved
...eb/nifi-web-frontend/src/main/nifi/src/app/pages/provenance/state/lineage/lineage.effects.ts
Outdated
Show resolved
Hide resolved
...eb/nifi-web-frontend/src/main/nifi/src/app/pages/provenance/state/lineage/lineage.effects.ts
Outdated
Show resolved
Hide resolved
...nance/ui/provenance-event-listing/provenance-event-table/provenance-event-table.component.ts
Outdated
Show resolved
Hide resolved
...s/provenance/ui/provenance-event-listing/provenance-event-table/lineage/lineage.component.ts
Outdated
Show resolved
Hide resolved
...s/provenance/ui/provenance-event-listing/provenance-event-table/lineage/lineage.component.ts
Outdated
Show resolved
Hide resolved
...s/provenance/ui/provenance-event-listing/provenance-event-table/lineage/lineage.component.ts
Outdated
Show resolved
Hide resolved
...nce/ui/provenance-event-listing/provenance-event-table/provenance-event-table.component.html
Outdated
Show resolved
Hide resolved
- Addressing review feedback.
Thanks for the review @rfellows! I've addressed the items you've mentioned and fixed the rendering issues as well. Please have a look at the latest commit. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just 2 things and then i think we are good.
...nce/ui/provenance-event-listing/provenance-event-table/provenance-event-table.component.html
Outdated
Show resolved
Hide resolved
...s/provenance/ui/provenance-event-listing/provenance-event-table/lineage/lineage.component.ts
Show resolved
Hide resolved
- Addressing review feedback.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the changes, @mcgilman. Looks good.
NIFI-12485: