-
Notifications
You must be signed in to change notification settings - Fork 74
Bump performance-analyzer-commons to v2.1.1 #910
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bump performance-analyzer-commons to v2.1.1 #910
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #910 +/- ##
============================================
- Coverage 70.88% 69.80% -1.08%
- Complexity 421 491 +70
============================================
Files 49 54 +5
Lines 3125 3520 +395
Branches 194 228 +34
============================================
+ Hits 2215 2457 +242
- Misses 785 911 +126
- Partials 125 152 +27 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
925a6bc to
10c0fc6
Compare
|
Why there are jackson changes? |
jackson core and annotations have different minor versions in OpenSearch-3.5.0 snapshot. Since we're using the same variable for both, build fails with invalid version. Using the version as per 3.5 snapshot. |
Please update the description accordingly. Update the release notes as well. |
10c0fc6 to
6bdd55a
Compare
Signed-off-by: Ajay Raj Nelapudi <ajnelapu@amazon.com>
6bdd55a to
beeedbe
Compare
Seems to be picking 2.15 commit 9cb092f and failing. Latest diff of codecov succeeds. |
Description
Consuming performance-analyzer-commons 2.1.1 on JDK21 with all versions bumped for OpenSearch 3.5 release. Takes in the following changes for 3.5 release.
Jackson core and annotations have different minor versions in OpenSearch-3.5.0 snapshot. Since we're using the same variable for both, build fails with invalid version. Using the version as per 3.5 snapshot.
Related Issues
Resolves #[Issue number to be closed when this PR is merged]
Check List
--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.