eejbyfeldt opened a new pull request, #38282: URL: https://github.com/apache/spark/pull/38282
### What changes were proposed in this pull request? This moves look up of the `IO_COMPRESSION_LZ4_BLOCKSIZE` and `IO_COMPRESSION_SNAPPY_BLOCKSIZE` to the constructor instead of the method `compressedOutputStream`. This is already how it done for `IO_COMPRESSION_ZSTD_BUFFERSIZE` in `ZStdCompressionCodec`. ### Why are the changes needed? The cost of looking up the config is often insignificant, but there are cases like a large shuffle with high number of partitions and therefore lots of small streams where the overhead becomes measurable (seen as high as a couple of % of executor cpu time). ### Does this PR introduce _any_ user-facing change? No. ### How was this patch tested? Existing unit tests. -- 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]
