etseidl opened a new pull request, #8742: URL: https://github.com/apache/arrow-rs/pull/8742
# Which issue does this PR close? - Part of #5853. # Rationale for this change Following the recent improvements in Thrift decoding, the percentage of time spent decoding LEB128 encoded integers has increased. # What changes are included in this PR? This PR modifies the varint decoder to first test for integers that can be encoded in a single byte (using zig-zag encoding, the maximum int that can be encoded is 63). Many of the fields in the Parquet footer (including all enum values) will be in this range, so optimizing for this frequent occurrence makes sense. # Are these changes tested? Should be covered by existing tests # Are there any user-facing changes? No -- 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]
