kgyrtkirk commented on PR #19248: URL: https://github.com/apache/druid/pull/19248#issuecomment-4175089687
@gianm: the version updates here are mostly to satisfy the [requireUpperBoundDeps](https://maven.apache.org/enforcer/enforcer-rules/requireUpperBoundDeps.html) - so that its less likely that a lib is forced to work with a library which is earlier than its requirement. Enabling it fully will help in the future to also update not-so-hot-topic libs as well in. That's why not necessarily the latest version is being used....I could file further updates after this PR but it could get into a loop that a newer version of a lib needs another lib to be upgraded which will cause issues... There is also an additional benefit of these changes like it also fix some issues of the release artifact - like we shipped 36.0 with 3 versions of snakeyaml: ``` apache-druid-36.0.0/extensions/druid-avro-extensions/snakeyaml-2.0.jar apache-druid-36.0.0/extensions/druid-kubernetes-extensions/snakeyaml-2.2.jar apache-druid-36.0.0/extensions/druid-kubernetes-overlord-extensions/snakeyaml-2.4.jar ``` -- 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]
