You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@calcite.apache.org by el...@apache.org on 2016/05/16 21:24:01 UTC

calcite git commit: [CALCITE-1236] Log a debug message when an error is sent back to the client

Repository: calcite
Updated Branches:
  refs/heads/master 0d2eae5be -> a15aa88bb


[CALCITE-1236] Log a debug message when an error is sent back to the client


Project: http://git-wip-us.apache.org/repos/asf/calcite/repo
Commit: http://git-wip-us.apache.org/repos/asf/calcite/commit/a15aa88b
Tree: http://git-wip-us.apache.org/repos/asf/calcite/tree/a15aa88b
Diff: http://git-wip-us.apache.org/repos/asf/calcite/diff/a15aa88b

Branch: refs/heads/master
Commit: a15aa88bb2820d8de008375d9033fc5c22fc1775
Parents: 0d2eae5
Author: Josh Elser <el...@apache.org>
Authored: Mon May 16 17:23:06 2016 -0400
Committer: Josh Elser <el...@apache.org>
Committed: Mon May 16 17:23:06 2016 -0400

----------------------------------------------------------------------
 .../java/org/apache/calcite/avatica/server/AvaticaJsonHandler.java  | 1 +
 .../org/apache/calcite/avatica/server/AvaticaProtobufHandler.java   | 1 +
 2 files changed, 2 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/calcite/blob/a15aa88b/avatica/server/src/main/java/org/apache/calcite/avatica/server/AvaticaJsonHandler.java
----------------------------------------------------------------------
diff --git a/avatica/server/src/main/java/org/apache/calcite/avatica/server/AvaticaJsonHandler.java b/avatica/server/src/main/java/org/apache/calcite/avatica/server/AvaticaJsonHandler.java
index b322d66..b639183 100644
--- a/avatica/server/src/main/java/org/apache/calcite/avatica/server/AvaticaJsonHandler.java
+++ b/avatica/server/src/main/java/org/apache/calcite/avatica/server/AvaticaJsonHandler.java
@@ -129,6 +129,7 @@ public class AvaticaJsonHandler extends AbstractAvaticaHandler {
             jsonResponse = jsonHandler.apply(jsonRequest);
           }
         } catch (Exception e) {
+          LOG.debug("Error invoking request from {}", baseRequest.getRemoteAddr(), e);
           jsonResponse = jsonHandler.convertToErrorResponse(e);
         }
 

http://git-wip-us.apache.org/repos/asf/calcite/blob/a15aa88b/avatica/server/src/main/java/org/apache/calcite/avatica/server/AvaticaProtobufHandler.java
----------------------------------------------------------------------
diff --git a/avatica/server/src/main/java/org/apache/calcite/avatica/server/AvaticaProtobufHandler.java b/avatica/server/src/main/java/org/apache/calcite/avatica/server/AvaticaProtobufHandler.java
index 7b08b0b..f8723f1 100644
--- a/avatica/server/src/main/java/org/apache/calcite/avatica/server/AvaticaProtobufHandler.java
+++ b/avatica/server/src/main/java/org/apache/calcite/avatica/server/AvaticaProtobufHandler.java
@@ -124,6 +124,7 @@ public class AvaticaProtobufHandler extends AbstractAvaticaHandler {
             handlerResponse = pbHandler.apply(requestBytes);
           }
         } catch (Exception e) {
+          LOG.debug("Error invoking request from {}", baseRequest.getRemoteAddr(), e);
           // Catch at the highest level of exceptions
           handlerResponse = pbHandler.convertToErrorResponse(e);
         }