You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by mi...@apache.org on 2014/03/19 18:05:29 UTC

svn commit: r1579299 - in /lucene/dev/branches/lucene5376_2/lucene/server/src: java/org/apache/lucene/server/Server.java test/org/apache/lucene/server/TestSearch.java

Author: mikemccand
Date: Wed Mar 19 17:05:29 2014
New Revision: 1579299

URL: http://svn.apache.org/r1579299
Log:
LUCENE-5376: add missing cause for some exceptions

Modified:
    lucene/dev/branches/lucene5376_2/lucene/server/src/java/org/apache/lucene/server/Server.java
    lucene/dev/branches/lucene5376_2/lucene/server/src/test/org/apache/lucene/server/TestSearch.java

Modified: lucene/dev/branches/lucene5376_2/lucene/server/src/java/org/apache/lucene/server/Server.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/lucene5376_2/lucene/server/src/java/org/apache/lucene/server/Server.java?rev=1579299&r1=1579298&r2=1579299&view=diff
==============================================================================
--- lucene/dev/branches/lucene5376_2/lucene/server/src/java/org/apache/lucene/server/Server.java (original)
+++ lucene/dev/branches/lucene5376_2/lucene/server/src/java/org/apache/lucene/server/Server.java Wed Mar 19 17:05:29 2014
@@ -442,7 +442,12 @@ public class Server {
       PrintWriter pw = new PrintWriter(sw);
       if (e.getCause() instanceof RequestFailedException) {
         RequestFailedException rfe = (RequestFailedException) e.getCause();
-        pw.write(rfe.path + ": " + rfe.reason);
+        pw.println(rfe.path + ": " + rfe.reason);
+        if (rfe.getCause() != null) {
+          pw.println();
+          rfe.getCause().printStackTrace(pw);
+        }
+        
         // TODO?
         //Throwable cause = rfe.getCause();
         //if (cause != null) {

Modified: lucene/dev/branches/lucene5376_2/lucene/server/src/test/org/apache/lucene/server/TestSearch.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/lucene5376_2/lucene/server/src/test/org/apache/lucene/server/TestSearch.java?rev=1579299&r1=1579298&r2=1579299&view=diff
==============================================================================
--- lucene/dev/branches/lucene5376_2/lucene/server/src/test/org/apache/lucene/server/TestSearch.java (original)
+++ lucene/dev/branches/lucene5376_2/lucene/server/src/test/org/apache/lucene/server/TestSearch.java Wed Mar 19 17:05:29 2014
@@ -358,4 +358,10 @@ public class TestSearch extends ServerBa
       send("startIndex");
     }
   }
+
+  public void testFailsQueryParserError() throws Exception {
+    assertFailsWith("search",
+                    "{queryText: 'field:bar', retrieveFields: [id],}",
+                    "field \"field\" is unknown");
+  }
 }