You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@asterixdb.apache.org by mh...@apache.org on 2018/04/30 20:51:17 UTC

asterixdb git commit: [NO ISSUE][OTH] Return Encountered Exception to NC

Repository: asterixdb
Updated Branches:
  refs/heads/master 4ed0eac5c -> f2eae8292


[NO ISSUE][OTH] Return Encountered Exception to NC

- user model changes: no
- storage format changes: no
- interface changes: no

Details:
- Return the encountered exception during query
  execution to NC instead of wrapping only the
  exception message in a new exception.

Change-Id: Ic51f3cba13edd4ef73c3833841e3990c2ed8fdfa
Reviewed-on: https://asterix-gerrit.ics.uci.edu/2628
Sonar-Qube: Jenkins <je...@fulliautomatix.ics.uci.edu>
Tested-by: Jenkins <je...@fulliautomatix.ics.uci.edu>
Contrib: Jenkins <je...@fulliautomatix.ics.uci.edu>
Reviewed-by: Murtadha Hubail <mh...@apache.org>
Integration-Tests: Jenkins <je...@fulliautomatix.ics.uci.edu>
Reviewed-by: Michael Blow <mb...@apache.org>


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

Branch: refs/heads/master
Commit: f2eae82923bf27f4485c318b6031f2836ac152b7
Parents: 4ed0eac
Author: Murtadha Hubail <mh...@apache.org>
Authored: Mon Apr 30 18:17:59 2018 +0300
Committer: Murtadha Hubail <mh...@apache.org>
Committed: Mon Apr 30 13:47:12 2018 -0700

----------------------------------------------------------------------
 .../apache/asterix/app/message/ExecuteStatementRequestMessage.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/asterixdb/blob/f2eae829/asterixdb/asterix-app/src/main/java/org/apache/asterix/app/message/ExecuteStatementRequestMessage.java
----------------------------------------------------------------------
diff --git a/asterixdb/asterix-app/src/main/java/org/apache/asterix/app/message/ExecuteStatementRequestMessage.java b/asterixdb/asterix-app/src/main/java/org/apache/asterix/app/message/ExecuteStatementRequestMessage.java
index d295304..8de3782 100644
--- a/asterixdb/asterix-app/src/main/java/org/apache/asterix/app/message/ExecuteStatementRequestMessage.java
+++ b/asterixdb/asterix-app/src/main/java/org/apache/asterix/app/message/ExecuteStatementRequestMessage.java
@@ -139,7 +139,7 @@ public final class ExecuteStatementRequestMessage implements ICcAddressedMessage
             responseMsg.setError(pe);
         } catch (Exception e) {
             GlobalConfig.ASTERIX_LOGGER.log(Level.ERROR, "Unexpected exception", e);
-            responseMsg.setError(new Exception(e.toString()));
+            responseMsg.setError(e);
         }
         try {
             messageBroker.sendApplicationMessageToNC(responseMsg, requestNodeId);