FrankChen021 commented on PR #20180: URL: https://github.com/apache/druid/pull/20180#issuecomment-5448037380
Backward compatibility: INCOMPATIBLE — verified at PR head 5e3836b9f5133a2b02a94c93b2a7d3f34fcac89c against upstream master 8c05f05adbe8f7ccf647c135caa948cc083fae8b. The PR changes only services/pom.xml but sets maven-resolver-impl to 2.0.22 while resolver API, SPI, util, connector-basic, and transport-http remain at 1.3.1. Pre-existing PullDependencies.java imports DefaultServiceLocator, which is absent from Resolver 2.0.22, and the 22 terminal CI failures confirm the resulting compile and RequireUpperBoundDeps breaks. Closure rationale: CLOSED_HIGH_EFFORT / WONT_FIX_LARGE_CHANGE. A valid Resolver 2.x update requires coordinated alignment of the Resolver family and Maven provider/transport, replacement of DefaultServiceLocator, service provisioning and session-lifecycle migration, license metadata updates, and CLI/distribution verification. This is a broad compatibility migration, not a smallest surgical fix. Code-change explanation: no tracked Java production or test source changed; the original PR changes only the maven-resolver-impl version line in services/pom.xml, and the failing PullDependencies.java call site is pre-existing and untouched. -- 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]
