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: 3a12e6214c5f45477042374ae3a4a569835d1fda / Xinyuan Lin <[email protected]> test(workflow-operator): add unit test coverage for interval-join, and sleep (#6098) ### What changes were proposed in this PR? Add unit test coverage for two `workflow-operator` executors, selected from the Codecov report. No production-code changes. | File | Codecov before | New coverage | | --- | --- | --- | | `IntervalJoinOpExec.scala` | 81% | extends the existing spec to cover the YEAR/MONTH/HOUR/MINUTE/SECOND timestamp interval-unit arms and the unsupported-key-type `WorkflowRuntimeException` | | `SleepOpExec.scala` | 0% | new spec: construction from a serialized desc + the pass-through `processTuple` (sleepTime 0) | The `IntervalJoin` `case None` interval arm requires a JSON payload with the field absent (Jackson maps an explicit `None` to `Some(null)`), so it is left to integration. > Note: an earlier revision of this PR also added a `CacheSourceOpExecSpec`. `CacheSourceOpExec` turned out to be dead code — cache source operators were replaced by input-port materialization reader threads in #3425, and its only factory (`SpecialPhysicalOpFactory.newSourcePhysicalOp`) has no production caller — so that spec was dropped and removal of the operator is tracked in #6125. ### Any related issues, documentation, discussions? Follow-up to the review feedback on #6043: prioritize tests that fill uncovered code paths. ### How was this PR tested? - `sbt "WorkflowOperator/testOnly *IntervalOpExecSpec *SleepOpExecSpec"` — all green - `sbt "WorkflowOperator/Test/scalafmtCheck"` and `sbt "WorkflowOperator/scalafixAll --check"` — clean ### Was this PR authored or co-authored using generative AI tooling? Generated-by: Claude Code (Opus 4.8 [1M context]) Report URL: https://github.com/apache/texera/actions/runs/28726007716 With regards, GitHub Actions via GitBox
