miinhho opened a new pull request, #5326:
URL: https://github.com/apache/zeppelin/pull/5326

   ### What is this PR for?
   This PR updates `SparkInterpreterLauncher` to throw 
`IllegalArgumentException` when the Scala version parsed from `spark-submit 
--version` is outside the supported range.
   
   Unsupported Scala versions are invalid input values for this validation 
path, so `IllegalArgumentException` better describes the failure than a generic 
`Exception`.
   
   The exception message is intentionally unchanged, and the 
`detectSparkScalaVersion(...)` method signature still declares `throws 
Exception` because the method can still raise checked exceptions from process 
execution, stream reading, and fallback Scala version detection.
   
   This is behavior-preserving for callers: the existing caller catches 
`Exception` and wraps it in an `IOException`, so the surfaced behavior and 
public API remain unchanged.
   
   ### What type of PR is it?
   Improvement
   
   ### What is the Jira issue?
   * https://issues.apache.org/jira/browse/ZEPPELIN-6464
   
   ### How should this be tested?
   * Build and run the module tests:
   ```  
   ./mvnw test -pl zeppelin-server --am
   ```
   
   ### Questions:
   * Does the license files need to update? No
   * Is there breaking changes for older versions? No
   * Does this needs documentation? No
   


-- 
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]

Reply via email to