You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@asterixdb.apache.org by mb...@apache.org on 2018/03/01 21:01:07 UTC

asterixdb git commit: [NO ISSUE][RT] Only suppress exception if not the same exception

Repository: asterixdb
Updated Branches:
  refs/heads/master 503c62200 -> 098360634


[NO ISSUE][RT] Only suppress exception if not the same exception

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

Details:
- Only suppress an exception if not the same as the original
  exception to avoid self suppression exceptions.

Change-Id: I2da6918128b4c1935d5092a4e7df36e0b80fde61
Reviewed-on: https://asterix-gerrit.ics.uci.edu/2445
Reviewed-by: Michael Blow <mb...@apache.org>
Sonar-Qube: Jenkins <je...@fulliautomatix.ics.uci.edu>
Tested-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/09836063
Tree: http://git-wip-us.apache.org/repos/asf/asterixdb/tree/09836063
Diff: http://git-wip-us.apache.org/repos/asf/asterixdb/diff/09836063

Branch: refs/heads/master
Commit: 09836063419486242fd396858bd6108b133ac10a
Parents: 503c622
Author: Abdullah Alamoudi <ba...@gmail.com>
Authored: Thu Mar 1 11:19:14 2018 -0800
Committer: Michael Blow <mb...@apache.org>
Committed: Thu Mar 1 13:00:49 2018 -0800

----------------------------------------------------------------------
 .../main/java/org/apache/hyracks/api/util/ExceptionUtils.java    | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/asterixdb/blob/09836063/hyracks-fullstack/hyracks/hyracks-api/src/main/java/org/apache/hyracks/api/util/ExceptionUtils.java
----------------------------------------------------------------------
diff --git a/hyracks-fullstack/hyracks/hyracks-api/src/main/java/org/apache/hyracks/api/util/ExceptionUtils.java b/hyracks-fullstack/hyracks/hyracks-api/src/main/java/org/apache/hyracks/api/util/ExceptionUtils.java
index 1a88e46..444b08f 100644
--- a/hyracks-fullstack/hyracks/hyracks-api/src/main/java/org/apache/hyracks/api/util/ExceptionUtils.java
+++ b/hyracks-fullstack/hyracks/hyracks-api/src/main/java/org/apache/hyracks/api/util/ExceptionUtils.java
@@ -102,7 +102,9 @@ public class ExceptionUtils {
         } else if (second == null) {
             return first;
         }
-        first.addSuppressed(second);
+        if (first != second) {
+            first.addSuppressed(second);
+        }
         return first;
     }
 }