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

Optimize NodeIndicesStats output behind flag #14454

Refactor validation and simplify tests

d78c948
Select commit
Loading
Failed to load commit list.
Merged

Optimize NodeIndicesStats output behind flag #14454

Refactor validation and simplify tests
d78c948
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Aug 22, 2024 in 0s

71.79% (target 70.00%)

View this Pull Request on Codecov

71.79% (target 70.00%)

Details

Codecov Report

Attention: Patch coverage is 81.25000% with 18 lines in your changes missing coverage. Please review.

Project coverage is 71.79%. Comparing base (0040f4b) to head (d78c948).
Report is 134 commits behind head on main.

Files Patch % Lines
.../java/org/opensearch/indices/NodeIndicesStats.java 85.36% 7 Missing and 5 partials ⚠️
...in/java/org/opensearch/indices/IndicesService.java 25.00% 2 Missing and 1 partial ⚠️
...h/action/admin/indices/stats/CommonStatsFlags.java 75.00% 0 Missing and 2 partials ⚠️
...rg/opensearch/rest/action/cat/RestNodesAction.java 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main   #14454      +/-   ##
============================================
+ Coverage     71.70%   71.79%   +0.08%     
- Complexity    62395    62504     +109     
============================================
  Files          5146     5146              
  Lines        293351   293426      +75     
  Branches      42412    42431      +19     
============================================
+ Hits         210352   210663     +311     
+ Misses        65645    65457     -188     
+ Partials      17354    17306      -48     

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