zhengruifeng opened a new pull request, #57886: URL: https://github.com/apache/spark/pull/57886
### What changes were proposed in this pull request? Snapshot the configured n-gram size while creating NGram's transform function. The resulting function no longer captures the NGram transformer. The serialized transform function decreases from 2,988 bytes to 1,277 bytes (57.3%). ### Why are the changes needed? Avoiding the transformer capture reduces the memory retained by the transform closure, which is especially useful for Spark Connect server workloads. ### Does this PR introduce _any_ user-facing change? No. ### How was this patch tested? Compiled the edited NGram source and serialized its generated transform function with Java serialization. The existing NGramSuite covers the unchanged transform behavior. The targeted SBT suite could not run locally because dependency resolution for `at.yawk:lz4-java:1.11.2` was denied by the configured Maven mirror. ### Was this patch authored or co-authored using generative AI tooling? Generated-by: Codex (GPT-5) -- 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]
