Skip to content

[DRAFT] Bump jackson from 2.14.2 to 2.15.0#16032

Closed
ssu2-atl wants to merge 1 commit intoopensearch-project:1.3from
ssu2-atl:16014-bump-jackson-to-2.15.0-from-2.14.2-on-opensearch-1.3
Closed

[DRAFT] Bump jackson from 2.14.2 to 2.15.0#16032
ssu2-atl wants to merge 1 commit intoopensearch-project:1.3from
ssu2-atl:16014-bump-jackson-to-2.15.0-from-2.14.2-on-opensearch-1.3

Conversation

@ssu2-atl
Copy link

@ssu2-atl ssu2-atl commented Sep 22, 2024

Description

Bump Jackson to 2.15.0 from 2.14.2 to address #16014

Check List

  • Functionality includes testing.
  • API changes companion pull request created, if applicable.
  • Public documentation issue/PR created, if applicable.

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.

@github-actions
Copy link
Contributor

❌ Gradle check result for eac8940: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@ssu2-atl
Copy link
Author

FAILURE: Build failed with an exception.

* What went wrong:
Failed to create Jar file /home/ubuntu/.gradle/caches/jars-8/adbff556ab863913e2be3ecb5dd121b1/jackson-core-2.15.0.jar.
> Unsupported class file major version 61

Think it's due to FasterXML/jackson-core#955

@opensearch-trigger-bot
Copy link
Contributor

This PR is stalled because it has been open for 30 days with no activity.

@opensearch-trigger-bot opensearch-trigger-bot bot added the stalled Issues that have stalled label Oct 24, 2024
@sandeshkr419
Copy link
Member

sandeshkr419 commented Nov 26, 2024

Jackson is already upgraded to 2.17.2 - closing this out.

Optionally, you may want to upgrade this to latest 2.18.1 if you are interested?

Correcting my response here: Jackson 2.15.0 may not be compatible with gradle version in OS 1.3.x and we won't be upgrading gradle in 1.3.x in my best of knowledge. So to mitigate #16014. we will have to safeguard OpenSearch without having to upgrade jackson.

@m-jin
Copy link

m-jin commented Dec 22, 2024

Jackson is already upgraded to 2.17.2 - closing this out.

Optionally, you may want to upgrade this to latest 2.18.1 if you are intrested?

@sandeshkr419 This is a PR for targeting Opensearch 1.3.x which is still on 2.14.2. You are talking about Jackson 2.17.2 on Opensearch 2.x.

@sandeshkr419 sandeshkr419 reopened this Dec 27, 2024
@sandeshkr419
Copy link
Member

@m-jin Yeah I was mistaken earlier, reopening it back.

@github-actions
Copy link
Contributor

❌ Gradle check result for eac8940: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@opensearch-trigger-bot opensearch-trigger-bot bot removed the stalled Issues that have stalled label Dec 27, 2024
@sandeshkr419
Copy link
Member

Closing this as OS 1.3 has reached end of maintenance window.

#19306

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