Skip to content

Remove guava from dependencies to fix jar hell issue#638

Merged
rupal-bq merged 1 commit intoopensearch-project:mainfrom
rupal-bq:jar-hell-issue
Feb 2, 2023
Merged

Remove guava from dependencies to fix jar hell issue#638
rupal-bq merged 1 commit intoopensearch-project:mainfrom
rupal-bq:jar-hell-issue

Conversation

@rupal-bq
Copy link
Copy Markdown
Contributor

@rupal-bq rupal-bq commented Feb 2, 2023

Signed-off-by: Rupal Mahajan maharup@amazon.com

Description

  • Changed mockito-core version to 5.1.0 to resolve conflicts
 > Conflict(s) found for the following module(s):
14 actionable tasks: 14 executed
          - org.mockito:mockito-core between versions 5.1.0 and 4.7.0
        Run with:
            --scan or
            :dependencyInsight --configuration testCompileClasspath --dependency org.mockito:mockito-core
  • Removed guava from dependencies to resolve
Caused by: java.lang.IllegalStateException: jar hell!
class: com.google.common.annotations.Beta
jar1: /tmp/tmpincskegu/opensearch-3.0.0/plugins/.installing-14976970994818525932/guava-31.0.1-jre.jar
jar2: /tmp/tmpincskegu/opensearch-3.0.0/plugins/opensearch-job-scheduler/guava-31.0.1-jre.jar
	at org.opensearch.bootstrap.JarHell.checkClass(JarHell.java:316)

Issues Resolved

#637

Check List

  • New functionality includes testing.
    • All tests pass, including unit test, integration test and doctest
  • New functionality has been documented.
    • New functionality has javadoc added
    • New functionality has user manual doc added
  • Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Signed-off-by: Rupal Mahajan <maharup@amazon.com>
@rupal-bq rupal-bq requested a review from a team as a code owner February 2, 2023 19:45
@rupal-bq
Copy link
Copy Markdown
Contributor Author

rupal-bq commented Feb 2, 2023

BWC tests are failing with

Execution failed for task ':reportsSchedulerBwcCluster#mixedClusterTask'.
> `cluster{::reportsSchedulerBwcCluster0}` failed to wait for cluster health yellow after 40 SECONDS
    IO error while waiting cluster
    503 Service Unavailable

Not related to this change

@rupal-bq rupal-bq merged commit 94d8cbb into opensearch-project:main Feb 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants