etseidl commented on PR #8190: URL: https://github.com/apache/arrow-rs/pull/8190#issuecomment-3224205460
> > .now the benchmark is down to 584us. > > Given the original description, does that mean that the orignal time was 1.75 ms (thus this PR is like a 3X improvement)? > > > open(page index) time: [1.7446 ms 1.7524 ms 1.7639 ms] The 1.75ms is the time for the decoder currently in main. The first phase of the remodel got that down to 700us, and this PR gets the time down to 585us (so a further 16% improvement). When combined with the column index changes in #8191 this benchmark is down to under 290us. -- 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]
