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: 531d79ff226acba9d5f8d1d13d57564359ba0cec / Matthew B. <[email protected]> fix: require non-null uid in insertNewExecution (#5300) ### What changes were proposed in this PR? - `ExecutionsMetadataPersistService.insertNewExecution` kept its `uid: Option[Integer]` signature but replaced `setUid(uid.orNull)` with `setUid(uid.getOrElse(throw new IllegalArgumentException(...)))`, so a missing uid is rejected with a clear error instead of writing null into the NOT NULL `workflow_executions.uid` column (which produced a cryptic jOOQ DataAccessException). - No `.orNull` is used; the caller (`WorkflowService.initExecutionService`) passes the `Option` through unchanged. - Updated `ExecutionsMetadataPersistServiceSpec`: the missing-uid case now passes `None` and asserts an `IllegalArgumentException` is raised. ### Any related issues, documentation, or discussions? Closes: #5212 ### How was this PR tested? - Compiled amber main + test sources under JDK 17 (clean). - Ran `WorkflowExecutionService/testOnly ...ExecutionsMetadataPersistServiceSpec`: all 9 tests pass. - Ran `sbt scalafmtAll` (clean). ### Was this PR authored or co-authored using generative AI tooling? Co-authored with Claude Opus 4.8 in compliance with ASF --------- Signed-off-by: Matthew B. <[email protected]> Report URL: https://github.com/apache/texera/actions/runs/26788927439 With regards, GitHub Actions via GitBox
