abdullah alamoudi has uploaded a new change for review.

  https://asterix-gerrit.ics.uci.edu/2445

Change subject: [NO ISSUE][RT] Only suppress exception if not the same exception
......................................................................

[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
---
M 
hyracks-fullstack/hyracks/hyracks-api/src/main/java/org/apache/hyracks/api/util/ExceptionUtils.java
1 file changed, 3 insertions(+), 1 deletion(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/45/2445/1

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 @@
         } else if (second == null) {
             return first;
         }
-        first.addSuppressed(second);
+        if (first != second) {
+            first.addSuppressed(second);
+        }
         return first;
     }
 }

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2445
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I2da6918128b4c1935d5092a4e7df36e0b80fde61
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: abdullah alamoudi <bamou...@gmail.com>

Reply via email to