OAK-12093: Rationalize dependency management#2729
OAK-12093: Rationalize dependency management#2729bhabegger wants to merge 1 commit intoapache:trunkfrom
Conversation
oak-parent/pom.xml
Outdated
| <guava.version>33.5.0-jre</guava.version> | ||
| <gson.version>2.11.0</gson.version> | ||
|
|
||
| <commons.compress.version>1.28.0</commons.compress.version> |
There was a problem hiding this comment.
Why do we need this here? I don't see why we would to introduce variables.
There was a problem hiding this comment.
So, here, I guess the convention is to have variables only when they have multiple uses ? Correct ?
reschke
left a comment
reschke
left a comment
There was a problem hiding this comment.
Some of these are good, but IMHO do not need variables.
Some of them are critical, when they affect embedded transitive dependencies.
This needs to be broken down into more separate changes.
| <scope>test</scope> | ||
| </dependency> | ||
|
|
||
| <!-- Guava and related dependencies --> |
There was a problem hiding this comment.
Nope, we are removing Guava.
There was a problem hiding this comment.
Is the removal of guava merged or soon to be ? If so I'll rebase and skip this then ;)
Any in particular you see as critical that should be taken care of separately ? |
bhabegger
changed the title
bhabegger
force-pushed
the
rationalize-dependencies
branch
from
79f9a5c to
a13d46a
Compare
| <dependency> | ||
| <groupId>org.testcontainers</groupId> | ||
| <artifactId>elasticsearch</artifactId> | ||
| <version>${testcontainers.version}</version> | ||
| </dependency> | ||
| <dependency> | ||
| <groupId>org.testcontainers</groupId> | ||
| <artifactId>mongodb</artifactId> | ||
| <version>${testcontainers.version}</version> | ||
| </dependency> | ||
| <dependency> | ||
| <groupId>org.testcontainers</groupId> | ||
| <artifactId>testcontainers</artifactId> | ||
| <version>${testcontainers.version}</version> | ||
| </dependency> |
There was a problem hiding this comment.
I don't think these should be in the parent pom but rather in the specific modules.
3 participants
This PR proposes to rationalize the dependencies in OAK. Some artifacts are downloaded in duplicates of different versions and even some times more.
To reduce these downloads, this PR removes explicit versions in sub-modules (unless strictly required, e.g. lucene for itself or lucene for elastic). All dependency versioning is centralized in the parent pom.
Number of downloads before : 999
Number of downloads after : 953