FrankChen021 commented on PR #19828: URL: https://github.com/apache/druid/pull/19828#issuecomment-5137958051
Follow-up: the standalone inline CodeQL comment in 8d5505714f was not associated with this annotated local declaration, so the check still reported alert 11448. I verified CodeQL Java suppression behavior against its official AlertSuppression tests and moved the exact rule ID into the declaration-level SuppressWarnings annotation in 0f4e0ad33f. CryptoServiceTest again passes 9/9 and git diff --check passes. The PR remains draft while the new current-head pipeline runs. -- 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]
