FrankChen021 commented on PR #19816: URL: https://github.com/apache/druid/pull/19816#issuecomment-5137176556
Addressed the latest Copilot findings in commit 39c3a1d864. ServerWrapper is now AutoCloseable, the max-frame, min-frame, and async Avatica tests close it on every exit path, their executors shut down in finally blocks, and the redundant nested CompressionUtils output-stream resource was removed. Targeted tests passed (3 Avatica tests and 1 CompressionUtils test), as did full processing/sql validation with Checkstyle and PMD. -- 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]
