Skip to content

[2.19] Update to Jackson 2.18.6 - #20813

Merged
andrross merged 1 commit into
opensearch-project:2.19from
andrross:jackson-2.18.6
Mar 9, 2026
Merged

[2.19] Update to Jackson 2.18.6#20813
andrross merged 1 commit into
opensearch-project:2.19from
andrross:jackson-2.18.6

Conversation

@andrross

@andrross andrross commented Mar 9, 2026

Copy link
Copy Markdown
Member

Check List

  • Functionality includes testing.

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.

@andrross
andrross requested a review from a team as a code owner March 9, 2026 19:57
@andrross andrross changed the title Update to Jackson 2.18.6 [2.19] Update to Jackson 2.18.6 Mar 9, 2026
@github-actions

github-actions Bot commented Mar 9, 2026

Copy link
Copy Markdown
Contributor

PR Reviewer Guide 🔍

(Review updated until commit c5026c9)

Here are some key observations to aid the review process:

🧪 No relevant tests
🔒 No security concerns identified
✅ No TODO sections
🔀 No multiple PR themes
⚡ Recommended focus areas for review

SHA1 Verification

The SHA1 checksums for the new Jackson 2.18.6 JARs should be verified against the official Maven Central repository to ensure the artifacts are authentic and have not been tampered with. Multiple modules share the same SHA1 for jackson-annotations and jackson-databind, which is expected, but should be confirmed.

jackson           = "2.18.6"
jackson_databind  = "2.18.6"

Signed-off-by: Andrew Ross <andrross@amazon.com>
@github-actions

github-actions Bot commented Mar 9, 2026

Copy link
Copy Markdown
Contributor

PR Code Suggestions ✨

Explore these optional code suggestions:

CategorySuggestion                                                                                                                                    Impact
General
Add missing PR reference in changelog

The changelog entry has an empty link with no PR number or URL. This should
reference the actual pull request that introduced this change, similar to other
entries in the changelog.

CHANGELOG.md [13]

-- Bump Jackson from 2.18.2 to 2.18.6 ([]())
+- Bump Jackson from 2.18.2 to 2.18.6 ([#XXXXX](https://github.com/opensearch-project/OpenSearch/pull/XXXXX))
Suggestion importance[1-10]: 6

__

Why: The changelog entry - Bump Jackson from 2.18.2 to 2.18.6 ([]()) has an empty link placeholder with no PR number or URL, unlike all other entries in the changelog. This should be filled in with the actual PR reference before merging.

Low

@github-actions

github-actions Bot commented Mar 9, 2026

Copy link
Copy Markdown
Contributor

Persistent review updated to latest commit c5026c9

@github-actions

github-actions Bot commented Mar 9, 2026

Copy link
Copy Markdown
Contributor

✅ Gradle check result for c5026c9: SUCCESS

@andrross
andrross merged commit 8735c32 into opensearch-project:2.19 Mar 9, 2026
44 checks passed
@andrross
andrross deleted the jackson-2.18.6 branch March 9, 2026 21:24
@codecov

codecov Bot commented Mar 9, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 71.95%. Comparing base (57ee1be) to head (c5026c9).
⚠️ Report is 3 commits behind head on 2.19.

Additional details and impacted files
@@             Coverage Diff              @@
##               2.19   #20813      +/-   ##
============================================
+ Coverage     71.90%   71.95%   +0.04%     
- Complexity    65964    66011      +47     
============================================
  Files          5342     5342              
  Lines        307362   307362              
  Branches      44857    44857              
============================================
+ Hits         221019   221152     +133     
+ Misses        67860    67728     -132     
+ Partials      18483    18482       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

2 participants