etseidl opened a new pull request, #11923: URL: https://github.com/apache/arrow-rs/pull/11923
# Which issue does this PR close? None as yet. # Rationale for this change AI identified a gap in the Thrift skip code where skipping a `list<bool>` would error. An initial fix had a serious perf regression. This PR exists to get a second opinion on a fix for that regression. # What changes are included in this PR? Properly skip boolean lists, but do so in a way that doesn't # Are these changes tested? Yes, perhaps too well # Are there any significant user-facing changes? No, this is a minor correctness fix that changes no public APIs. In practice this should have no effect because the only bool list in the current parquet IDL is in the column index, which we do not skip via this path. -- 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]
