aglinxinyuan opened a new issue, #8895:
URL: https://github.com/apache/texera/issues/8895

   ### Task Summary
   
   `execute-workflow.service.ts` exports three REST endpoint constants that 
nothing has used since execution moved to the websocket: 
`EXECUTE_WORKFLOW_ENDPOINT` (`queryplan/execute`), `PAUSE_WORKFLOW_ENDPOINT` 
(`pause`) and `RESUME_WORKFLOW_ENDPOINT` (`resume`).
   
   **Introduced by:**
   
   - #622 (2018-06-21, the first execute-workflow service) added 
`EXECUTE_WORKFLOW_ENDPOINT`.
   - #684 (2019-03-10, pause/resume buttons) added the other two.
   
   **Usage removed by:**
   
   - #811 (2020-08-21, the Amber merge) moved pause and resume to websocket 
requests, removing their `http.post` calls.
   - #982 (2021-01-25, "Improves multiple ports support in Texera operators 
(Step 4)") removed the last REST execute call.
   
   Execute, pause and resume now go over the websocket as 
`WorkflowExecuteRequest` / `WorkflowPauseRequest` / `WorkflowResumeRequest`, 
and the backend has no matching REST routes.
   
   This is a pure deletion with no behaviour change.
   
   ### Task Type
   
   - [x] Refactor / Cleanup
   


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