Akanksha-kedia opened a new pull request, #18958:
URL: https://github.com/apache/pinot/pull/18958
## Summary
`RangeIndexHandler.existingRangeIndexVersionDiffers()` detects v1↔v2 on-disk
version mismatches that require an index rebuild (v1 = `RangeIndexCreator`, v2
= `BitSlicedRangeIndexCreator` — incompatible layouts and different query
semantics), but this logic had no test coverage.
- Adds `RangeIndexHandlerTest` with 7 unit tests:
- v1 on disk + v2 configured → rebuild triggered
- v2 on disk + v1 configured → rebuild triggered (downgrade also requires
rebuild)
- Matching version → no rebuild
- Column removed from config → `removeIndex` called on writer
- New unsorted column added to config → needs index creation
- Sorted column in config → skipped (sorted columns don't use range index)
## Test plan
- [ ] `RangeIndexHandlerTest` — 7 unit tests, all green
- [ ] `./mvnw checkstyle:check -pl pinot-segment-local` passes (0 violations)
🤖 Generated with [Claude Code](https://claude.com/claude-code)
--
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]
