You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@livy.apache.org by "gyogal (via GitHub)" <gi...@apache.org> on 2026/03/02 05:00:01 UTC

Re: [PR] [LIVY-789] Ensure Spark data files are available in interactive session [incubator-livy]

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: commits-unsubscribe@livy.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org