xiangfu0 opened a new pull request, #19281: URL: https://github.com/apache/pinot/pull/19281
Extracted from #18229 as an independent correctness prerequisite. ForwardIndexHandler creates ForwardIndexReaderContext instances and PinotSegmentColumnReader wrappers while rewriting or collecting forward-index statistics. These resources can own direct buffers, but several paths did not close them. This change uses try-with-resources across those paths and preserves the existing reader and creator ownership boundaries. Tests: - ForwardIndexHandlerContextTest: closes the context exactly once on success and read failure - pinot-segment-local Spotless, Checkstyle, license format, and license check This PR does not add codec-pipeline behavior and can merge independently of the codec stack. -- 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]
