JeonDaehong commented on PR #18027: URL: https://github.com/apache/iceberg/pull/18027#issuecomment-5698214452
@pvary Thank you for guiding me! ``` [Daehong] Reducing per-row position delete index probes in the vectorized read path * Vectorized reads probe the position delete index once per row, but the positions in a batch are a contiguous ascending range. The index can be traversed once instead. * Significant gain when few columns are projected. Not visible end to end on wide projections, where the delete check is a small share of scan CPU. * Needs a new API: PositionDeleteIndex.forEachInRange. Is the community OK with that? * PR: #18027 ``` I wrote it this way. I hope it leads to a great discussion! As an engineer who loves Iceberg, I'm thrilled to be a part of this journey. -- 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]
