This is an automated email from the ASF dual-hosted git repository.

msingh pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/hadoop-ozone.git


The following commit(s) were added to refs/heads/master by this push:
     new 29cd37d  HDDS-2280. HddsUtils#CheckForException should not return null 
in case the ratis exception cause is not set. Contributed by  Shashikant 
Banerjee. (#57)
29cd37d is described below

commit 29cd37db9f712844403f1d0bb5af94e2d72712a5
Author: bshashikant <shashik...@apache.org>
AuthorDate: Sun Oct 20 16:50:12 2019 +0530

    HDDS-2280. HddsUtils#CheckForException should not return null in case the 
ratis exception cause is not set. Contributed by  Shashikant Banerjee. (#57)
---
 .../main/java/org/apache/hadoop/hdds/scm/client/HddsClientUtils.java | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git 
a/hadoop-hdds/client/src/main/java/org/apache/hadoop/hdds/scm/client/HddsClientUtils.java
 
b/hadoop-hdds/client/src/main/java/org/apache/hadoop/hdds/scm/client/HddsClientUtils.java
index d3bb31a..e1262b4 100644
--- 
a/hadoop-hdds/client/src/main/java/org/apache/hadoop/hdds/scm/client/HddsClientUtils.java
+++ 
b/hadoop-hdds/client/src/main/java/org/apache/hadoop/hdds/scm/client/HddsClientUtils.java
@@ -304,9 +304,12 @@ public final class HddsClientUtils {
     return scmSecurityClient;
   }
 
+  // This will return the underlying exception after unwrapping
+  // the exception to see if it matches with expected exception
+  // list otherwise will return the exception back.
   public static Throwable checkForException(Exception e) {
     Throwable t = e;
-    while (t != null) {
+    while (t != null && t.getCause() != null) {
       for (Class<? extends Exception> cls : getExceptionList()) {
         if (cls.isInstance(t)) {
           return t;


---------------------------------------------------------------------
To unsubscribe, e-mail: hdfs-commits-unsubscr...@hadoop.apache.org
For additional commands, e-mail: hdfs-commits-h...@hadoop.apache.org

Reply via email to