gyogal commented on code in PR #503:
URL: https://github.com/apache/incubator-livy/pull/503#discussion_r2870510908
##########
server/src/main/scala/org/apache/livy/utils/SparkYarnApp.scala:
##########
@@ -155,8 +153,9 @@ class SparkYarnApp private[utils] (
// We cannot kill the YARN app without the app id.
// There's a chance the YARN app hasn't been submitted during a
livy-server failure.
// We don't want a stuck session that can't be deleted. Emit a warning
and move on.
- case _: TimeoutException | _: InterruptedException =>
+ case e @ (_: InterruptedException | NonFatal(_)) =>
Review Comment:
Thanks for working on this! The tests now look good, but I am wondering why
the LIVY-1016 commit is showing in the PR (this file and the one below)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
