Skip to content

[Backport 2.x] Adding support to register settings dynamically#5544

Merged
saratvemulapalli merged 1 commit into2.xfrom
backport/backport-5495-to-2.x
Dec 13, 2022
Merged

[Backport 2.x] Adding support to register settings dynamically#5544
saratvemulapalli merged 1 commit into2.xfrom
backport/backport-5495-to-2.x

Conversation

@opensearch-trigger-bot
Copy link
Copy Markdown
Contributor

Backport 08cd06f from #5495.

* Adding support to register settings dynamically

Signed-off-by: Ryan Bogan <rbogan@amazon.com>

* Update CHANGELOG

Signed-off-by: Ryan Bogan <rbogan@amazon.com>

* Removed unnecessary registerSetting methods

Signed-off-by: Ryan Bogan <rbogan@amazon.com>

* Change setting registration order

Signed-off-by: Ryan Bogan <rbogan@amazon.com>

* Add unregisterSettings method

Signed-off-by: Ryan Bogan <rbogan@amazon.com>

* Remove unnecessary feature flag

Signed-off-by: Ryan Bogan <rbogan@amazon.com>

Signed-off-by: Ryan Bogan <rbogan@amazon.com>
(cherry picked from commit 08cd06f)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@saratvemulapalli saratvemulapalli added backport PRs or issues specific to backporting features or enhancments v2.5.0 'Issues and PRs related to version v2.5.0' enhancement Enhancement or improvement to existing feature or request labels Dec 13, 2022
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Dec 13, 2022

Codecov Report

Merging #5544 (74afe1e) into 2.x (816e12c) will decrease coverage by 0.13%.
The diff coverage is 67.74%.

@@             Coverage Diff              @@
##                2.x    #5544      +/-   ##
============================================
- Coverage     70.71%   70.58%   -0.14%     
+ Complexity    58492    58420      -72     
============================================
  Files          4716     4716              
  Lines        279676   279705      +29     
  Branches      40809    40819      +10     
============================================
- Hits         197785   197425     -360     
- Misses        65478    65821     +343     
- Partials      16413    16459      +46     
Impacted Files Coverage Δ
...search/common/settings/AbstractScopedSettings.java 88.16% <33.33%> (-1.33%) ⬇️
...org/opensearch/common/settings/SettingsModule.java 85.51% <81.81%> (-0.67%) ⬇️
...java/org/opensearch/client/indices/DataStream.java 0.00% <0.00%> (-76.09%) ⬇️
...adonly/AddIndexBlockClusterStateUpdateRequest.java 0.00% <0.00%> (-75.00%) ⬇️
...adcast/BroadcastShardOperationFailedException.java 33.33% <0.00%> (-66.67%) ⬇️
...port/ResponseHandlerFailureTransportException.java 0.00% <0.00%> (-60.00%) ⬇️
...readonly/TransportVerifyShardIndexBlockAction.java 9.75% <0.00%> (-41.47%) ⬇️
...indices/readonly/TransportAddIndexBlockAction.java 20.68% <0.00%> (-37.94%) ⬇️
...java/org/opensearch/threadpool/ThreadPoolInfo.java 56.25% <0.00%> (-37.50%) ⬇️
...nsearch/index/shard/IndexShardClosedException.java 66.66% <0.00%> (-33.34%) ⬇️
... and 492 more

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

@github-actions
Copy link
Copy Markdown
Contributor

Gradle Check (Jenkins) Run Completed with:

@saratvemulapalli saratvemulapalli merged commit 0ea42f2 into 2.x Dec 13, 2022
@github-actions github-actions bot deleted the backport/backport-5495-to-2.x branch December 13, 2022 18:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport PRs or issues specific to backporting features or enhancments enhancement Enhancement or improvement to existing feature or request v2.5.0 'Issues and PRs related to version v2.5.0'

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants