Akanksha-kedia commented on PR #18920: URL: https://github.com/apache/pinot/pull/18920#issuecomment-4915087028
The only failing check is **Pinot Binary Compatibility Check**, which is pre-existing and unrelated to this PR. The failure is in `pinot-spi` — methods removed from `org.apache.pinot.spi.data.readers.ColumnReader` (`hasNext()`, `nextInt()`, `nextLong()`, etc.) and `PinotDataType.toUuidBytesArray()`. These are upstream changes by other contributors that already break binary compat on master. This PR only touches `pinot-segment-local` (JSON index rebuild logic and `JsonIndexConfig` equals/hashCode). All other checks pass: Linter ✅, Unit Tests ✅, Integration Tests ✅, Quickstart ✅, Compatibility Regression ✅. -- 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]
