You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by da...@apache.org on 2019/05/07 08:42:17 UTC
[lucene-solr] branch branch_8_1 updated: SOLR-13449:
SolrClientNodeStateProvider always retries on requesting metrics from other
nodes
This is an automated email from the ASF dual-hosted git repository.
datcm 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 fdcd093 SOLR-13449: SolrClientNodeStateProvider always retries on requesting metrics from other nodes
fdcd093 is described below
commit fdcd093490dd3526b0ca7dad5dbdb4f6ed4cbb46
Author: Cao Manh Dat <da...@apache.org>
AuthorDate: Tue May 7 09:40:45 2019 +0100
SOLR-13449: SolrClientNodeStateProvider always retries on requesting metrics from other nodes
---
solr/CHANGES.txt | 2 ++
.../org/apache/solr/client/solrj/impl/SolrClientNodeStateProvider.java | 2 ++
2 files changed, 4 insertions(+)
diff --git a/solr/CHANGES.txt b/solr/CHANGES.txt
index 22c8221..7da8f47 100644
--- a/solr/CHANGES.txt
+++ b/solr/CHANGES.txt
@@ -209,6 +209,8 @@ 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 f164484..a1f507c 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,6 +199,7 @@ 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;
@@ -305,6 +306,7 @@ 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) {