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

Posted to reviews@zeppelin.apache.org by "renechoi (via GitHub)" <gi...@apache.org> on 2026/08/31 12:07:29 UTC

Re: [PR] [ZEPPELIN-6258] Improve Process resource management in SparkInterpreterLauncher [zeppelin]

renechoi commented on PR #4998:
URL: https://github.com/apache/zeppelin/pull/4998#issuecomment-5478088321

   Sorry for the long delay, and rebased now. #5000 landed in the meantime and rewrote `detectSparkScalaVersion`, so replaying the old diff would have undone that; this is the same fix written against the current code.
   
   Added `testDetectSparkScalaVersionWithLargeProcessOutput`, which points SPARK_HOME at a fake spark-submit writing 256KB to stdout. It hangs on master and passes here.
   


-- 
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: reviews-unsubscribe@zeppelin.apache.org

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