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] Add file cache stats, reservation logic #6485

Merged
merged 2 commits into from
Feb 25, 2023

Conversation

kotwanikunal
Copy link
Member

Description

Issues Resolved

  • N/A

Check List

  • New functionality includes testing.
    • All tests pass
  • New functionality has been documented.
    • New functionality has javadoc added
  • Commits are signed per the DCO using --signoff
  • Commit changes are listed out in CHANGELOG.md file (See: Changelog)

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.

adnapibar and others added 2 commits February 24, 2023 15:50
)

This change exposes the file cache stats as part the node stats API response.

Signed-off-by: Rabi Panda <adnapibar@gmail.com>
(cherry picked from commit 0f3b870)
Signed-off-by: Kunal Kotwani <kkotwani@amazon.com>
(cherry picked from commit eb78246)
@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@codecov-commenter
Copy link

Codecov Report

Merging #6485 (9354f73) into 2.x (46caaec) will increase coverage by 0.06%.
The diff coverage is 63.67%.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@             Coverage Diff              @@
##                2.x    #6485      +/-   ##
============================================
+ Coverage     70.27%   70.34%   +0.06%     
- Complexity    59100    59271     +171     
============================================
  Files          4796     4797       +1     
  Lines        284245   284416     +171     
  Branches      41310    41328      +18     
============================================
+ Hits         199750   200069     +319     
+ Misses        67767    67614     -153     
- Partials      16728    16733       +5     
Impacted Files Coverage Δ
...min/cluster/stats/TransportClusterStatsAction.java 70.83% <ø> (ø)
...search/cluster/MockInternalClusterInfoService.java 0.00% <0.00%> (ø)
.../java/org/opensearch/test/InternalTestCluster.java 58.33% <12.50%> (-0.84%) ⬇️
...rch/action/admin/cluster/node/stats/NodeStats.java 48.57% <22.22%> (-1.43%) ⬇️
...rc/main/java/org/opensearch/monitor/fs/FsInfo.java 70.40% <31.25%> (-2.68%) ⬇️
...src/main/java/org/opensearch/node/NodeService.java 70.31% <33.33%> (-0.66%) ⬇️
...org/opensearch/common/settings/SettingsModule.java 83.55% <40.00%> (-0.91%) ⬇️
...c/main/java/org/opensearch/index/IndexService.java 73.87% <62.50%> (-0.68%) ⬇️
.../main/java/org/opensearch/env/NodeEnvironment.java 76.26% <68.85%> (-0.69%) ⬇️
...h/index/store/remote/filecache/FileCacheStats.java 71.18% <71.18%> (ø)
... and 520 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants