swyxio opened a new pull request, #16539: URL: https://github.com/apache/pinot/pull/16539
Implemented column-major indexing flow in SegmentIndexCreationDriver: 1. Added `PinotSegmentRecordReader.getIndexSegment()` – exposes the underlying segment to column-major consumers. 2. Introduced `buildIndexesByColumn(...)` – single internal helper that performs column-oriented indexing given that stats are already collected. 3. Refactored the public `buildByColumn(...)` to call the helper (maintaining external API). 4. In `build()`, if the input is a `PinotSegmentRecordReader`, we now delegate to the new column-major path and return immediately – eliminating the second full row-scan while reusing collected stats. 5. Removed obsolete duplicate code. No linter errors after the changes. -- 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]
