This is an automated email from the ASF dual-hosted git repository. github-merge-queue[bot] pushed a commit to branch gh-readonly-queue/main/pr-8896-535b97a941131cb0ac3a57f112b4e4c60f059d7e in repository https://gitbox.apache.org/repos/asf/texera.git
commit a42c573f7d4515fbaa4dcecbb4399b01829310ef Author: Xinyuan Lin <[email protected]> AuthorDate: Thu Oct 8 07:40:39 2026 +0000 refactor(frontend): remove unused workflow endpoint constants (#8896) ### What changes were proposed in this PR? Removes three exported REST endpoint constants from `execute-workflow.service.ts` that nothing imports: `EXECUTE_WORKFLOW_ENDPOINT`, `PAUSE_WORKFLOW_ENDPOINT` and `RESUME_WORKFLOW_ENDPOINT`. **−5 lines.** There is no behaviour change. Execute, pause and resume are sent over the websocket (`WorkflowExecuteRequest` / `WorkflowPauseRequest` / `WorkflowResumeRequest`), and the backend has no `queryplan/execute`, `pause` or `resume` REST routes. ### History | | | | --- | --- | | **Introduced by** | #622 (2018-06-21), the first execute-workflow service: `EXECUTE_WORKFLOW_ENDPOINT`. #684 (2019-03-10), pause/resume buttons: the other two | | **Usage removed by** | #811 (2020-08-21), the Amber merge, which moved pause/resume to websocket requests and removed their `http.post` calls. #982 (2021-01-25), "Improves multiple ports support in Texera operators (Step 4)", which removed the last REST execute call | The pause/resume constants have been unused for six years, and the execute one for nearly five. ### Any related issues, documentation, discussions? Closes #8895 ### How was this PR tested? No new tests; no spec imported the constants. From `frontend/`: - `npx ng test --watch=false`, covering `execute-workflow.service.spec.ts` plus all 17 specs that import the service (18 files): 1363 tests pass, 1 is skipped (an existing `it.skip`). - `yarn format:ci`: clean. - `npx ng build`: success. To re-check: ``` git grep -n -w -E "EXECUTE_WORKFLOW_ENDPOINT|PAUSE_WORKFLOW_ENDPOINT|RESUME_WORKFLOW_ENDPOINT" # no hits ``` ### Was this PR authored or co-authored using generative AI tooling? Generated-by: Claude Code (Claude Opus 5.5) 🤖 Generated with [Claude Code](https://claude.com/claude-code) --- .../workspace/service/execute-workflow/execute-workflow.service.ts | 5 ----- 1 file changed, 5 deletions(-) diff --git a/frontend/src/app/workspace/service/execute-workflow/execute-workflow.service.ts b/frontend/src/app/workspace/service/execute-workflow/execute-workflow.service.ts index a321381681..c513ebe7c6 100644 --- a/frontend/src/app/workspace/service/execute-workflow/execute-workflow.service.ts +++ b/frontend/src/app/workspace/service/execute-workflow/execute-workflow.service.ts @@ -55,11 +55,6 @@ import { GuiConfigService } from "../../../common/service/gui-config.service"; // TODO: change this declaration export const FORM_DEBOUNCE_TIME_MS = 150; -export const EXECUTE_WORKFLOW_ENDPOINT = "queryplan/execute"; - -export const PAUSE_WORKFLOW_ENDPOINT = "pause"; -export const RESUME_WORKFLOW_ENDPOINT = "resume"; - /** * ExecuteWorkflowService sends the current workflow data to the backend * for execution, then receives backend's response and broadcast it to other components.
