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

Revert "[Backport 2.2] Handles the status code for . properties" #4276

Merged
merged 1 commit into from
Aug 23, 2022

Conversation

owaiskazi19
Copy link
Member

Reverts #4251

@owaiskazi19 owaiskazi19 requested a review from dblock August 22, 2022 18:37
@owaiskazi19 owaiskazi19 requested review from a team and reta as code owners August 22, 2022 18:37
This reverts commit fafdd28.

Signed-off-by: Owais Kazi <owaiskazi19@gmail.com>
@owaiskazi19 owaiskazi19 force-pushed the revert-4251-backport/backport-4246-to-2.2 branch from 4fe8941 to cb91a69 Compare August 22, 2022 18:39
@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@codecov-commenter
Copy link

Codecov Report

Merging #4276 (4fe8941) into 2.2 (b1017fa) will increase coverage by 0.07%.
The diff coverage is 100.00%.

❗ Current head 4fe8941 differs from pull request most recent head cb91a69. Consider uploading reports for the commit cb91a69 to get more accurate results

@@             Coverage Diff              @@
##                2.2    #4276      +/-   ##
============================================
+ Coverage     70.51%   70.59%   +0.07%     
- Complexity    56679    56697      +18     
============================================
  Files          4557     4557              
  Lines        272873   272874       +1     
  Branches      40055    40055              
============================================
+ Hits         192423   192625     +202     
+ Misses        64275    64059     -216     
- Partials      16175    16190      +15     
Impacted Files Coverage Δ
server/src/main/java/org/opensearch/Version.java 82.54% <100.00%> (+0.55%) ⬆️
...adcast/BroadcastShardOperationFailedException.java 33.33% <0.00%> (-66.67%) ⬇️
.../java/org/opensearch/node/NodeClosedException.java 50.00% <0.00%> (-50.00%) ⬇️
...ava/org/opensearch/action/NoSuchNodeException.java 0.00% <0.00%> (-50.00%) ⬇️
...pensearch/indices/breaker/CircuitBreakerStats.java 27.77% <0.00%> (-41.67%) ⬇️
.../admin/cluster/reroute/ClusterRerouteResponse.java 60.00% <0.00%> (-40.00%) ⬇️
...search/aggregations/metrics/ParsedPercentiles.java 42.10% <0.00%> (-38.95%) ⬇️
...luster/routing/allocation/RoutingExplanations.java 62.06% <0.00%> (-37.94%) ⬇️
...search/aggregations/pipeline/HoltWintersModel.java 21.47% <0.00%> (-31.55%) ⬇️
...cluster/routing/allocation/RerouteExplanation.java 70.00% <0.00%> (-30.00%) ⬇️
... and 473 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
Contributor

Gradle Check (Jenkins) Run Completed with:

@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@kartg kartg merged commit 1a1ffa3 into 2.2 Aug 23, 2022
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.

5 participants