FrankChen021 commented on PR #20277: URL: https://github.com/apache/druid/pull/20277#issuecomment-5582660719
Loop 2 ownership review at exact head b02e206b614c50d787faf03896b64b60b8c086aa (base ref 9b90983fd291f26935af934383ce360473179e4d). The PR change is limited to jetty.version 12.1.12 -> 12.1.13 and the matching licenses.yaml version. Compatibility result: SAFE for this bounded dependency update. Maven Central contains the full published Jetty 12.1 line through 12.1.13; this is the adjacent 12.1.12 -> 12.1.13 transition across the 10 managed Druid coordinates. Public japicmp comparison of all 10 source/target jars found no incompatible API/ABI changes. Druid Jetty server/client/proxy/rewrite/servlet usage and overrides were checked; release POM differences leave production dependencies unchanged apart from Jetty version alignment. The release additions are additive or bug fixes; SessionAuthentication persistence remains opt-in with the default false, and no Druid persistence, wire, serialization, client, license, or extension/plugin SPI change was found. CI is currently blocked by the moving-base merge context, not this PR: the failed CodeQL Java, Static Checks Maven/packaging/strict/openrewrite/web, Docker, all JDK25 shards, and all four QTest shards fail compiling current-master SupervisorManagerTest references (missing Assert, MatcherAssert, and DruidExceptionMatcher). The merge run is built on current master f37486d51b6e8ac21ca3a536b38826b7d6edf5bb, while the PR base ref remains 9b90983fd291f26935af934383ce360473179e4d. web-checks JavaScript tests passed before the same Maven compile failure; the R*,B*,Q*,V* shard had two retry-pass flaky tests but still ended on the deterministic compile failure. coverage-jacoco was downstream skipped. No PR-caused failure was found, so there is no bounded PR fix to push and no eligible rerun while the deterministic base mismatch remains. I am not approving or closing until the base failure is repaired/rebased and the exact-head required checks complete successfully. -- 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]
