PG1204 commented on code in PR #6213:
URL: https://github.com/apache/texera/pull/6213#discussion_r3888059392
##########
frontend/src/app/workspace/service/workflow-graph/model/joint-graph-wrapper.ts:
##########
@@ -108,6 +109,11 @@ export class JointGraphWrapper {
// reapply it to the shared model (covering both the main canvas and the
mini-map).
private regionsDisplayedStream = new BehaviorSubject<boolean>(false);
+ // The active performance heat-map view, or null when the overlay is off
(Layers > Performance).
+ // Kept here so the editor can (re)apply operator colors on the shared
model, covering both the
+ // main canvas and the mini-map.
+ private heatmapViewStream = new BehaviorSubject<HeatmapView | null>(null);
Review Comment:
WorkflowEditorComponent.ngOnDestroy now resets the view to null, and added a
spec pinning it so the checkbox, the stream, and the metrics all agree on
re-entry.
--
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]
