(zeppelin) branch master updated: [ZEPPELIN-6478] Log Groovy classpath discovery failures with throwable

[chanholee]

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 7dfc421f68 [ZEPPELIN-6478] Log Groovy classpath discovery failures 
with throwable
7dfc421f68 is described below

commit 7dfc421f686ce89eb5963fb10bd7eb37dfa75717
Author: 백형준 <138356797+vividb...@users.noreply.github.com>
AuthorDate: Sun Aug 2 16:36:51 2026 +0900

    [ZEPPELIN-6478] Log Groovy classpath discovery failures with throwable
    
    ### What is this PR for?
    
      Pass the caught exception to SLF4J when Groovy classpath discovery fails 
in `GroovyInterpreter.open()`.
    
      Previously, only `e.getMessage()` was logged, so the exception stack 
trace was lost. This change logs a descriptive message together with the 
throwable while preserving the existing non-fatal fallback behavior.
    
      ### What type of PR is it?
    
      Improvement
    
      ### Todos
    
      * [x] Pass the classpath discovery exception to SLF4J
      * [x] Preserve the existing non-fatal control flow
      * [x] Run the Groovy module test lifecycle
    
      ### What is the Jira issue?
    
      https://issues.apache.org/jira/browse/ZEPPELIN-6478
    
      ### How should this be tested?
    
      The following command completed successfully:
    
      * `./mvnw test -pl groovy`
        * Build succeeded
    
      ### Screenshots (if appropriate)
    
      N/A
    
      ### Questions:
    
      * Does the license files need to update? No
      * Is there breaking changes for older versions? No
      * Does this needs documentation? No
    
    Closes #5363 from vividbaek/ZEPPELIN-6478-log-groovy-classpath-failure.
    
    Signed-off-by: ChanHo Lee <chanho...@apache.org>
---
 groovy/src/main/java/org/apache/zeppelin/groovy/GroovyInterpreter.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/groovy/src/main/java/org/apache/zeppelin/groovy/GroovyInterpreter.java 
b/groovy/src/main/java/org/apache/zeppelin/groovy/GroovyInterpreter.java
index 3c6cbed874..f2a88ccf39 100644
--- a/groovy/src/main/java/org/apache/zeppelin/groovy/GroovyInterpreter.java
+++ b/groovy/src/main/java/org/apache/zeppelin/groovy/GroovyInterpreter.java
@@ -76,7 +76,7 @@ public class GroovyInterpreter extends Interpreter {
                 .getPath());
         classes = new File(jar.getParentFile(), "classes").toString();
       } catch (Exception e) {
-        LOGGER.error(e.getMessage());
+        LOGGER.error("Failed to resolve Groovy classpath", e);
       }
     }
     LOGGER.info("groovy classes classpath: " + classes);