You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by is...@apache.org on 2019/05/07 16:02:55 UTC
[lucene-solr] branch branch_8_1 updated: SOLR-13449: Reverting from
branch_8_1 due to test failures
This is an automated email from the ASF dual-hosted git repository.
ishan pushed a commit to branch branch_8_1
in repository https://gitbox.apache.org/repos/asf/lucene-solr.git
The following commit(s) were added to refs/heads/branch_8_1 by this push:
new e5839fb SOLR-13449: Reverting from branch_8_1 due to test failures
e5839fb is described below
commit e5839fb416083fcdaeedfb1e329a9fdaa29fdc50
Author: Ishan Chattopadhyaya <is...@apache.org>
AuthorDate: Tue May 7 21:32:40 2019 +0530
SOLR-13449: Reverting from branch_8_1 due to test failures
---
solr/CHANGES.txt | 2 --
.../org/apache/solr/client/solrj/impl/SolrClientNodeStateProvider.java | 2 --
2 files changed, 4 deletions(-)
diff --git a/solr/CHANGES.txt b/solr/CHANGES.txt
index 7da8f47..22c8221 100644
--- a/solr/CHANGES.txt
+++ b/solr/CHANGES.txt
@@ -209,8 +209,6 @@ Bug Fixes
* SOLR-13318: Fix ClassCastException in SolrJ JsonFaceting classes (Munendra S N via Jason Gerlowski)
-* SOLR-13449: SolrClientNodeStateProvider always retries on requesting metrics from other nodes (Cao Manh Dat)
-
Improvements
----------------------
diff --git a/solr/solrj/src/java/org/apache/solr/client/solrj/impl/SolrClientNodeStateProvider.java b/solr/solrj/src/java/org/apache/solr/client/solrj/impl/SolrClientNodeStateProvider.java
index a1f507c..f164484 100644
--- a/solr/solrj/src/java/org/apache/solr/client/solrj/impl/SolrClientNodeStateProvider.java
+++ b/solr/solrj/src/java/org/apache/solr/client/solrj/impl/SolrClientNodeStateProvider.java
@@ -199,7 +199,6 @@ public class SolrClientNodeStateProvider implements NodeStateProvider, MapWriter
while (cnt++ < 3) {
try {
rsp = ctx.invoke(solrNode, CommonParams.METRICS_PATH, params);
- break;
} catch (SolrException | SolrServerException | IOException e) {
boolean hasCauseIOException = false;
Throwable cause = e;
@@ -306,7 +305,6 @@ public class SolrClientNodeStateProvider implements NodeStateProvider, MapWriter
while (cnt++ < retries) {
try {
rsp = snitchContext.invoke(solrNode, CommonParams.METRICS_PATH, params);
- break;
} catch (SolrException | SolrServerException | IOException e) {
if (e instanceof SolrServerException) {