Ma77Ball opened a new pull request, #5387: URL: https://github.com/apache/texera/pull/5387
### What changes were proposed in this PR? - Add `AmberKryoInitializerSpec`, the first test for the Kryo closure-serialization setup that lets Pekko move Scala closures between workers. - Assert `preInit` registers `SerializedLambda` and binds `ClosureSerializer.Closure` to a `ClosureSerializer`, with a negative control proving a bare `ScalaKryo` has neither. - Add behavioral round-trips: a Scala closure serializes and deserializes back to a working function, and two closures capturing different locals stay independent. ### Any related issues, documentation, or discussions? N/A ### How was this PR tested? - Run `sbt "WorkflowExecutionService/testOnly *AmberKryoInitializerSpec"` and expect 4 examples to pass. - Local sbt test runs are blocked by JaCoCo 0.8.11 (cannot instrument Java 25 bytecode, major version 69), so the suite was run directly via ScalaTest off the dependency classpath (`java -cp ... org.scalatest.tools.Runner -s ...AmberKryoInitializerSpec` with `--add-opens java.base/java.lang.invoke=ALL-UNNAMED`); all 4 passed. CI runs it on a supported JDK. ### 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]
