FrankChen021 opened a new pull request, #19821: URL: https://github.com/apache/druid/pull/19821
Created by the GPT-5.6-Sol model. ## Summary - enable PMD's `UnusedFormalParameter` and `UnusedLocalVariable` rules in the Maven static-check lifecycle - remove six unused parameters from private production methods and their call sites - remove two genuinely dead local variables - rename ten intentionally unused try-with-resources handles with PMD's supported `ignored*` convention The broader CodeQL pattern also includes generated sources, public/interface parameters, and test scaffolding. This change focuses the Maven gate on actionable handwritten production code without breaking compatibility APIs or try-with-resources cleanup semantics. ## Verification - `mvn -ntp -B pmd:check -Dweb.console.skip=true -DskipTests -T1C` - `mvn -ntp -B test-compile -pl processing,server,indexing-service,sql,extensions-core/google-extensions,extensions-contrib/opentelemetry-emitter,extensions-contrib/rabbit-stream-indexing-service -am -Dweb.console.skip=true -DskipTests -T1C` -- 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]
