You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by jd...@apache.org on 2012/03/20 18:40:40 UTC
svn commit: r1303037 - in /lucene/dev/branches/branch_3x/solr: ./
CHANGES.txt solrj/src/java/org/apache/solr/common/SolrException.java
Author: jdyer
Date: Tue Mar 20 17:40:40 2012
New Revision: 1303037
URL: http://svn.apache.org/viewvc?rev=1303037&view=rev
Log:
SOLR-2124: do not log stack trace for Service Disabled / 503
Modified:
lucene/dev/branches/branch_3x/solr/ (props changed)
lucene/dev/branches/branch_3x/solr/CHANGES.txt
lucene/dev/branches/branch_3x/solr/solrj/src/java/org/apache/solr/common/SolrException.java
Modified: lucene/dev/branches/branch_3x/solr/CHANGES.txt
URL: http://svn.apache.org/viewvc/lucene/dev/branches/branch_3x/solr/CHANGES.txt?rev=1303037&r1=1303036&r2=1303037&view=diff
==============================================================================
--- lucene/dev/branches/branch_3x/solr/CHANGES.txt (original)
+++ lucene/dev/branches/branch_3x/solr/CHANGES.txt Tue Mar 20 17:40:40 2012
@@ -254,7 +254,10 @@ Bug Fixes
package name (org.apache.commons.csv). This created a compatibility issue as
the Apache Commons team works toward an official release of Commons CSV.
JARJAR (http://code.google.com/p/jarjar/) was used to change the package name
- to org.apache.solr.internal.csv. (Uwe Schindler, Emmanuel Bourg)
+ to org.apache.solr.internal.csv. (Uwe Schindler, Emmanuel Bourg)
+
+* SOLR-2124: Do not log stack traces for "Service Disabled" / 503 Exceptions (PingRequestHandler, etc)
+ (James Dyer, others)
Other Changes
----------------------
Modified: lucene/dev/branches/branch_3x/solr/solrj/src/java/org/apache/solr/common/SolrException.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/branch_3x/solr/solrj/src/java/org/apache/solr/common/SolrException.java?rev=1303037&r1=1303036&r2=1303037&view=diff
==============================================================================
--- lucene/dev/branches/branch_3x/solr/solrj/src/java/org/apache/solr/common/SolrException.java (original)
+++ lucene/dev/branches/branch_3x/solr/solrj/src/java/org/apache/solr/common/SolrException.java Tue Mar 20 17:40:40 2012
@@ -150,6 +150,7 @@ public class SolrException extends Runti
public static void log(Logger log, Throwable e) {
if (e instanceof SolrException) {
((SolrException)e).logged = true;
+ if (((SolrException) e).code() == ErrorCode.SERVICE_UNAVAILABLE.code) return;
}
String stackTrace = toStr(e);
String ignore = doIgnore(stackTrace);
@@ -164,6 +165,7 @@ public class SolrException extends Runti
public static void log(Logger log, String msg, Throwable e) {
if (e instanceof SolrException) {
((SolrException)e).logged = true;
+ if (((SolrException) e).code() == ErrorCode.SERVICE_UNAVAILABLE.code) return;
}
String stackTrace = msg + ':' + toStr(e);
String ignore = doIgnore(stackTrace);
@@ -180,6 +182,7 @@ public class SolrException extends Runti
public static void logOnce(Logger log, String msg, Throwable e) {
if (e instanceof SolrException) {
if(((SolrException)e).logged) return;
+ if (((SolrException) e).code() == ErrorCode.SERVICE_UNAVAILABLE.code) return;
}
if (msg!=null) log(log,msg,e);
else log(log,e);