Yicong-Huang commented on code in PR #8301:
URL: https://github.com/apache/texera/pull/8301#discussion_r3922288849


##########
frontend/src/app/workspace/component/workflow-editor/workflow-editor.component.ts:
##########
@@ -1199,14 +1206,14 @@ export class WorkflowEditorComponent implements OnInit, 
AfterViewInit, OnDestroy
 
   /**
    * Applies the validation result to the operator's border. Delegates to
-   * applyOperatorBorder so validation, cached-execution-status, and the
+   * applyOperatorStateAndBorder so validation, cached-execution-status, and 
the

Review Comment:
   "cached-execution-status" is the pre-split umbrella word. The helper reads 
`getCurrentState()`, and its own doc at `:573` already says "cached execution 
state". My miss from the earlier rounds, not something this commit introduced.
   
   ```suggestion
      * applyOperatorStateAndBorder so validation, cached-execution-state, and 
the
   ```



-- 
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]

Reply via email to