Ma77Ball opened a new pull request, #5915: URL: https://github.com/apache/texera/pull/5915
### What changes were proposed in this PR? - Switch the two CSV-sourced tests in `ReconfigurationIntegrationSpec` from `smallCsvScanOpDesc` (100 rows) to `mediumCsvScanOpDesc` (100k rows), so the workflow is still running when `pauseWorkflow` lands. - This removes a timing race: with a 100-row source the run could reach COMPLETED before pause took effect, so `PauseHandler.pauseWorkflow` emitted COMPLETED instead of PAUSED and `TestUtils.shouldReconfigure`s 10s `pausedReached` await timed out. - Aligns the integration spec with its non-flaky siblings `ReconfigurationSpec` and `PauseSpec`, which already use `mediumCsvScanOpDesc` for the same pause/reconfigure/resume flow. ### Any related issues, documentation, discussions? Closes: #5913 ### How was this PR tested? - `sbt "WorkflowExecutionService/Test/compile"` compiles cleanly with the change. - Full run requires the `amber-integration` stack (Python UDF + Postgres/MinIO/Lakekeeper) that CI provisions; could not run locally. Reviewer: run the `amber-integration` job (or `sbt "WorkflowExecutionService/testOnly *ReconfigurationIntegrationSpec"` in that environment) and confirm all 3 tests pass across repeated runs. - Expect a `workflow paused` log line for every test run (the flaky run showed fewer pauses than test runs). ### Was this PR authored or co-authored using generative AI tooling? Co-authored with Claude Opus 4.8 in compliance with ASF -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
