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] Disallowing compression level for lz4 and best_compression codec #9060

Merged
merged 1 commit into from
Aug 2, 2023

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 82b5f3c from #8737.

* Disallowing compression level for lz4 and best_compression codec
* setting up codec settings interface for validation

Signed-off-by: Sarthak Aggarwal <sarthagg@amazon.com>
(cherry picked from commit 82b5f3c)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@github-actions
Copy link
Contributor

github-actions bot commented Aug 2, 2023

Gradle Check (Jenkins) Run Completed with:

  • RESULT: UNSTABLE ❕
  • TEST FAILURES:
      1 org.opensearch.search.SearchWeightedRoutingIT.testStrictWeightedRoutingWithCustomString

@codecov
Copy link

codecov bot commented Aug 2, 2023

Codecov Report

Merging #9060 (f89c785) into 2.x (d711f66) will decrease coverage by 0.18%.
The diff coverage is 80.00%.

@@             Coverage Diff              @@
##                2.x    #9060      +/-   ##
============================================
- Coverage     70.91%   70.74%   -0.18%     
+ Complexity    57458    57402      -56     
============================================
  Files          4749     4749              
  Lines        271376   271395      +19     
  Branches      40051    40055       +4     
============================================
- Hits         192453   191988     -465     
- Misses        62479    63010     +531     
+ Partials      16444    16397      -47     
Files Changed Coverage Δ
...java/org/opensearch/index/engine/EngineConfig.java 96.05% <77.77%> (-2.47%) ⬇️
...opensearch/index/codec/customcodecs/ZstdCodec.java 100.00% <100.00%> (ø)
...arch/index/codec/customcodecs/ZstdNoDictCodec.java 100.00% <100.00%> (+14.28%) ⬆️

... and 464 files with indirect coverage changes

@reta reta merged commit 558b56d into 2.x Aug 2, 2023
31 checks passed
@github-actions github-actions bot deleted the backport/backport-8737-to-2.x branch August 2, 2023 11:57
@sarthakaggarwal97
Copy link
Contributor

thanks @reta

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