The GitHub Actions job "Required Checks" on 
texera.git/gh-readonly-queue/main/pr-8835-3ba45ee5a5f641388d1a9eda90afa26a62d70d30
 has failed.
Run started by GitHub user aglinxinyuan (triggered by aglinxinyuan).

Head commit for run:
f916c96c53d75c6661505bb804c9c80557b9d716 / Xinyuan Lin <[email protected]>
refactor(workflow-operator): remove the unused toImmutableMap (#8835)

### What changes were proposed in this PR?

Removes `OperatorDescriptorUtils.toImmutableMap`, which no production
code calls. **−40 lines.** There is no behaviour change. The PR also
removes the `scala.jdk.CollectionConverters._` import, which only this
method used, and the three spec cases that only exercised it.
`equallyPartitionGoal`, the object's only live member, and its spec
cases are unchanged.

### History

| | |
| --- | --- |
| **Introduced by** | #2439 (2024-03-05), "Schema propagation push down
to PhysicalPlan", as `AmberUtils.toImmutableMap` for Java operator
descriptors. #2667 (2024-05-16) moved it into `OperatorDescriptorUtils`
|
| **Usage removed by** | #3179 (2024-12-29), "Convert Java operator
descriptors to Scala". It rewrote the last Java callers
(`TypeCastingOpDesc`, `PythonUDFSourceOpDescV2`, `RUDFSourceOpDesc`) in
Scala, which needs no conversion |

It has been dead for over twenty-one months.

### Any related issues, documentation, discussions?

Closes #8834

### How was this PR tested?

No new tests. The five `equallyPartitionGoal` cases remain.

Locally, from the repo root with Java 17:

- `sbt "WorkflowOperator/Test/compile"
"WorkflowExecutionService/Test/compile"`: success.
- `sbt "WorkflowOperator/testOnly *OperatorDescriptorUtilsSpec"`: 5
tests, all pass.
- `sbt "WorkflowOperator/scalafmtCheckAll" "WorkflowOperator/scalafixAll
--check"`: clean.

To re-check:

```
git grep -n "toImmutableMap"   # 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)

Report URL: https://github.com/apache/texera/actions/runs/39549871339

With regards,
GitHub Actions via GitBox

Reply via email to