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

jagadish pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/samza.git


The following commit(s) were added to refs/heads/master by this push:
     new 1322cb5  SAMZA-2165: [FOLLOW-UP] Ignore all exceptions from the AM 
during status
1322cb5 is described below

commit 1322cb5d0312be3aa36c568604a7645ba357f3e8
Author: Jagadish <jvenkatra...@linkedin.com>
AuthorDate: Wed Apr 17 17:52:51 2019 -0700

    SAMZA-2165: [FOLLOW-UP] Ignore all exceptions from the AM during status
    
    Author: Jagadish <jvenkatra...@linkedin.com>
    
    Reviewers: Abhishek S<ashiv...@linkedin.com>
    
    Closes #1000 from vjagadish1989/yarnjob-fix1
---
 samza-yarn/src/main/scala/org/apache/samza/job/yarn/ClientHelper.scala | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/samza-yarn/src/main/scala/org/apache/samza/job/yarn/ClientHelper.scala 
b/samza-yarn/src/main/scala/org/apache/samza/job/yarn/ClientHelper.scala
index c937523..4147e6f 100644
--- a/samza-yarn/src/main/scala/org/apache/samza/job/yarn/ClientHelper.scala
+++ b/samza-yarn/src/main/scala/org/apache/samza/job/yarn/ClientHelper.scala
@@ -351,7 +351,7 @@ class ClientHelper(conf: Configuration) extends Logging {
         false
       }
     } catch {
-      case e: IOException => {
+      case e: Exception => {
         // ignore any exceptions when querying the AM - likely due to YARN 
restarting the AM process
         warn("Exception when querying AM metrics", e)
         false

Reply via email to