etseidl commented on PR #11262: URL: https://github.com/apache/arrow-rs/pull/11262#issuecomment-6003297954
Not a huge difference, but it seems `DefinitionLevelDecoderImpl::skip_def_levels` can be improved some. It resizes its temp buffer, and then passes that to `self.read_def_levels`, which resizes it again. Initializing it's capacity at `SKIP_BUFFER_SIZE`, and then inlining the two lines of `read_def_levels` that we need, I was able to speed up the problematic round trip benches by around 10%. -- 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]
