Apache
/
lucene
mirror of https://github.com/apache/lucene.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

when sending a non retry update fails, log it at warn level 

git-svn-id: https://svn.apache.org/repos/asf/lucene/dev/trunk@1554109 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
Mark Robert Miller 2013-12-30 00:25:10 +00:00
parent f63357957e
commit e73c522f0b
1 changed files with 6 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
solr/core/src/java/org/apache/solr/update/processor/DistributedUpdateProcessor.java
View File

@ -614,6 +614,12 @@ public class DistributedUpdateProcessor extends UpdateRequestProcessor {
// if one node is a RetryNode, this was a forward request // if one node is a RetryNode, this was a forward request
if (errors.get(0).req.node instanceof RetryNode) { if (errors.get(0).req.node instanceof RetryNode) {
rsp.setException(errors.get(0).e); rsp.setException(errors.get(0).e);
} else {
if (log.isWarnEnabled()) {
for (Error error : errors) {
log.warn("Error sending update", error.e);
}
}
} }
// else // else
// for now we don't error - we assume if it was added locally, we // for now we don't error - we assume if it was added locally, we
@ -625,7 +631,6 @@ public class DistributedUpdateProcessor extends UpdateRequestProcessor {
// recover - the doc was already added locally, so it should have been // recover - the doc was already added locally, so it should have been
// legit // legit
// TODO: we should do this in the background it would seem
for (final SolrCmdDistributor.Error error : errors) { for (final SolrCmdDistributor.Error error : errors) {
if (error.req.node instanceof RetryNode) { if (error.req.node instanceof RetryNode) {
// we don't try to force a leader to recover // we don't try to force a leader to recover