xiangfu0 opened a new pull request, #19397: URL: https://github.com/apache/pinot/pull/19397
## Summary Extracts the codec-runtime changes requested in #19307 into an independently reviewable prerequisite. - Replaces the unreleased allocating handler API with required destination-based `encode` and `decode` methods; no compatibility adapters. - Uses bounded caller-owned encode/decode workspaces, with reusable intermediate decode views and immutable cached plans. - Migrates existing real-codec fixtures and adds focused destination, scratch lifecycle, corruption-bound, cache, and concurrency tests. No mocks. V7 reader/writer, routing, table-config validation, and reload integration remain in #19307. ## Validation - Fresh prerequisite-only checkout: 307 codec/parser tests pass, including 39 destination-contract cases and 31 executor cases. - Spotless, Checkstyle, license formatting, and license checks pass. - Optional warning-enabled compilation fails on both base `80dbb7d02d` and this change because dependency metadata references missing JetBrains `NotNull`; the normal build and tests pass. -- 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]
