You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tez.apache.org by rb...@apache.org on 2014/09/19 17:25:09 UTC

git commit: TEZ-1598. DAGClientTimelineImpl uses ReflectiveOperationException (which has JDK 1.7 dependency) (Rajesh Balamohan) (Cherry picked from commit c360b517394f858fa73d6737fd0dfbbeffe59c7f)

Repository: tez
Updated Branches:
  refs/heads/branch-0.5 4e085fa32 -> 6a46e11c6


TEZ-1598. DAGClientTimelineImpl uses ReflectiveOperationException (which has JDK 1.7 dependency) (Rajesh Balamohan)
(Cherry picked from commit c360b517394f858fa73d6737fd0dfbbeffe59c7f)


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

Branch: refs/heads/branch-0.5
Commit: 6a46e11c6a8057d5f713c7acacb63ffa299e38a5
Parents: 4e085fa
Author: Rajesh Balamohan <rb...@apache.org>
Authored: Fri Sep 19 20:51:07 2014 +0530
Committer: Rajesh Balamohan <rb...@apache.org>
Committed: Fri Sep 19 20:54:29 2014 +0530

----------------------------------------------------------------------
 CHANGES.txt                                              |  1 +
 .../apache/tez/dag/api/client/DAGClientTimelineImpl.java | 11 +++++++++--
 2 files changed, 10 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/tez/blob/6a46e11c/CHANGES.txt
----------------------------------------------------------------------
diff --git a/CHANGES.txt b/CHANGES.txt
index bb6231e..573b748 100644
--- a/CHANGES.txt
+++ b/CHANGES.txt
@@ -28,6 +28,7 @@ ALL CHANGES
   TEZ-1495. ATS integration for TezClient
   TEZ-1553. Multiple failures in testing path-related tests in
   TestTezCommonUtils for Windows
+  TEZ-1598. DAGClientTimelineImpl uses ReflectiveOperationException (which has JDK 1.7 dependency)
 
 Release 0.5.0: 2014-09-03
 

http://git-wip-us.apache.org/repos/asf/tez/blob/6a46e11c/tez-api/src/main/java/org/apache/tez/dag/api/client/DAGClientTimelineImpl.java
----------------------------------------------------------------------
diff --git a/tez-api/src/main/java/org/apache/tez/dag/api/client/DAGClientTimelineImpl.java b/tez-api/src/main/java/org/apache/tez/dag/api/client/DAGClientTimelineImpl.java
index 57453aa..6cbab80 100644
--- a/tez-api/src/main/java/org/apache/tez/dag/api/client/DAGClientTimelineImpl.java
+++ b/tez-api/src/main/java/org/apache/tez/dag/api/client/DAGClientTimelineImpl.java
@@ -21,6 +21,7 @@ package org.apache.tez.dag.api.client;
 import javax.annotation.Nullable;
 import javax.ws.rs.core.MediaType;
 import java.io.IOException;
+import java.lang.reflect.InvocationTargetException;
 import java.lang.reflect.Method;
 import java.net.HttpURLConnection;
 import java.net.URL;
@@ -448,8 +449,14 @@ public class DAGClientTimelineImpl extends DAGClient {
       Class<?> yarnConfiguration = Class.forName("org.apache.hadoop.yarn.conf.YarnConfiguration");
       final Method useHttps = yarnConfiguration.getMethod("useHttps", Configuration.class);
       return (Boolean)useHttps.invoke(null, conf);
-    } catch (ReflectiveOperationException e) {
-      throw new TezException("error accessing yarn configuration", e);
+    } catch (ClassNotFoundException e) {
+      throw new TezException(e);
+    } catch (InvocationTargetException e) {
+      throw new TezException(e);
+    } catch (NoSuchMethodException e) {
+      throw new TezException(e);
+    } catch (IllegalAccessException e) {
+      throw new TezException(e);
     }
   }