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