The GitHub Actions job "Required Checks" on texera.git/main has succeeded. Run started by GitHub user github-merge-queue[bot] (triggered by github-merge-queue[bot]).
Head commit for run: 557983acb436572b25d9121ae1234695a727e8a1 / Xinyuan Lin <[email protected]> chore(amber): remove the unused search-by-operators endpoint (#8331) ### What changes were proposed in this PR? Deletes `WorkflowResource.searchWorkflowByOperator` (`GET /workflow/search-by-operators`), which has no client. Pure deletion, no behaviour change: **−66 lines**. Workflow search in the UI goes to the unified `/dashboard/search` resource — the frontend's search service builds `${API}/dashboard/search`, and no file references this path. ### History | | | | --- | --- | | **Introduced by** | #1611 (2022-08-07) — "Search Workflows Feature" | | **Usage removed by** | #2038 (2023-07-07) — "Allow users to cascade the sharing" deleted the frontend constant `WORKFLOW_OPERATOR_URL = WORKFLOW_BASE_URL + "/search-by-operators"` when search moved to the dashboard resource | Dead for about three years; workflow search has gone to `/dashboard/search` ever since. > Reviewer note: the frontend's `searchByOperators` test in `user-workflow.component.spec.ts` is a name coincidence, not a caller — it drives the filter UI (`component.filters.operators`, `masterFilterList`), which queries `/dashboard/search`. Nothing in the frontend, or anywhere else, builds this URL. Removing the method also frees `Condition` and `noCondition`, which no other method in `WorkflowResource` uses — scalafix flagged both, and they are the only other lines touched. ### Any related issues, documentation, discussions? Closes #8328 ### How was this PR tested? Existing tests only — this PR adds none, since it removes an endpoint and the one test that covered it. Locally, from the repo root with Java 17: - `sbt "WorkflowExecutionService/Test/compile"` — success. - `sbt "WorkflowExecutionService/testOnly *WorkflowResourceSpec"` — 80 tests, all pass. - `sbt scalafmtCheckAll "scalafixAll --check"` — clean. Verification, re-runnable by a reviewer: ``` git grep -rn "search-by-operators\|searchWorkflowByOperator" # only the deleted method and its test git grep -rn "dashboard/search" -- frontend/src # where search actually goes ``` ### Was this PR authored or co-authored using generative AI tooling? Generated-by: Claude Code (Claude Opus 5) Report URL: https://github.com/apache/texera/actions/runs/36092458231 With regards, GitHub Actions via GitBox
