peter-toth opened a new pull request, #56988: URL: https://github.com/apache/spark/pull/56988
### What changes were proposed in this pull request? This is a follow-up build fix of https://github.com/apache/spark/pull/56970 (SPARK-57889) for `branch-3.5`. The cherry-pick of SPARK-57889 to `branch-3.5` fails to compile because the new `OneForOneStreamManagerSuite.testStreamRequestAuthorization` test uses `Assertions.assertThrows` (JUnit 5). On `master` this suite has already been migrated to JUnit 5, but on `branch-3.5` it still uses JUnit 4 (`org.junit.Assert` / `org.junit.Test`), so `Assertions` does not resolve. This changes the single call to JUnit 4's `Assert.assertThrows` (available since JUnit 4.13), consistent with the rest of the file on `branch-3.5`. No production code changes. ### Why are the changes needed? Without this fix, `common/network-common` test compilation fails on `branch-3.5`, breaking the build after the SPARK-57889 cherry-pick. ### Does this PR introduce _any_ user-facing change? No. Test-only change. ### How was this patch tested? `build/mvn -pl common/network-common surefire:test -Dtest=OneForOneStreamManagerSuite` now compiles and passes (5 tests run, 0 failures), including `testStreamRequestAuthorization`. ### Was this patch authored or co-authored using generative AI tooling? Generated-by: Claude Code (Opus 4.8) -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
