raghavyadav01 opened a new pull request, #16298: URL: https://github.com/apache/pinot/pull/16298
Fixed couple of issues in CLP V2 implementation and added integration tests. ## Issues Fixed ### 1. Magic Bytes Location CLP V2 was writing a length prefix before magic bytes, causing mismatch with reader factory expecting magic bytes at offset 0. Updated `CLPForwardIndexCreatorV2` and `CLPForwardIndexReaderV2` to write and read magic bytes directly at offset 0, matching V1's simpler format. ### 2. get Compression Codec API CLPForwardIndexReaderV2 did not implement getCompressionType and return value was null causing UpdateIndices to fail during segment load. ### 3. Dictionary File Location Bug Issue in `CLPForwardIndexCreatorV2` where dictionary variable forward index file was created with incorrect path. ## Testing - Added `CLPV2EncodingRealtimeIntegrationTest` that extends existing CLP test ## Impact - Fixes CLP V2 compression functionality . -- 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]
