Skip to content
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

[Backport 2.x] Bound the size of cache in deprecation logger (#16724) #16770

Merged

Conversation

andrross
Copy link
Member

@andrross andrross commented Dec 3, 2024

Backports b1bf72f from #16724

Copy link
Contributor

github-actions bot commented Dec 3, 2024

❌ Gradle check result for 6693fb7: 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?

CHANGELOG.md Outdated Show resolved Hide resolved
The current implementation of the map used to de-duplicate deprecation log
messages can grow without bound. This adds a simple fixed limit to the data
structure tracking existing loggers. Once the limit is breached new loggers will
no longer log deprecation warnings. I also added a check to skip the tracking
if the deprecation logger is disabled.

Signed-off-by: Andrew Ross <[email protected]>
(cherry picked from commit b1bf72f)
Signed-off-by: Andrew Ross <[email protected]>
@andrross andrross force-pushed the backport/backport-16724-to-2.x branch from 6693fb7 to 7449adb Compare December 4, 2024 19:15
Copy link
Contributor

github-actions bot commented Dec 4, 2024

❕ Gradle check result for 7449adb: UNSTABLE

Please review all flaky tests that succeeded after retry and create an issue if one does not already exist to track the flaky failure.

Copy link

codecov bot commented Dec 4, 2024

Codecov Report

Attention: Patch coverage is 91.66667% with 1 line in your changes missing coverage. Please review.

Project coverage is 71.89%. Comparing base (ff59e9c) to head (7449adb).
Report is 4 commits behind head on 2.x.

Files with missing lines Patch % Lines
...g/opensearch/common/logging/DeprecationLogger.java 75.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##                2.x   #16770      +/-   ##
============================================
- Coverage     71.94%   71.89%   -0.05%     
- Complexity    65473    65494      +21     
============================================
  Files          5314     5314              
  Lines        305297   305364      +67     
  Branches      44490    44501      +11     
============================================
- Hits         219648   219555      -93     
- Misses        67341    67528     +187     
+ Partials      18308    18281      -27     

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

@reta reta merged commit 6d28b4f into opensearch-project:2.x Dec 4, 2024
39 checks passed
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.

3 participants