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

jongyoul 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 30bb6a8367 [ZEPPELIN-6477] Stop dumping stack trace on Groovy 
cancellation
30bb6a8367 is described below

commit 30bb6a8367c2b75c37225f3d44a9d23bf9e63b8b
Author: 백형준 <[email protected]>
AuthorDate: Mon Aug 3 23:20:44 2026 +0900

    [ZEPPELIN-6477] Stop dumping stack trace on Groovy cancellation
    
    ### What is this PR for?
    
      Remove the `Thread.dumpStack()` call from Groovy paragraph cancellation 
because it writes an unnecessary stack trace directly to stderr.
    
      The existing `t.interrupt()` call is preserved, so the paragraph 
cancellation behavior remains unchanged.
    
      ### What type of PR is it?
    
      Improvement
    
      ### Todos
    
      * [x] Remove the unnecessary stack dump
      * [x] Preserve the thread interruption behavior
      * [x] Run the Groovy module build and tests
    
      ### What is the Jira issue?
    
      https://issues.apache.org/jira/browse/ZEPPELIN-6477
    
      ### How should this be tested?
    
      ```bash
      ./mvnw test -pl groovy
      ```
    
      The build completed successfully.
    
      ### Screenshots (if appropriate)
    
      Not applicable.
    
      ### Questions:
    
      - Does the license files need to update? No
      - Is there breaking changes for older versions? No
      - Does this needs documentation? No
    
    Closes #5364 from vividbaek/ZEPPELIN-6477-remove-groovy-cancel-stack-dump.
    
    Signed-off-by: Jongyoul Lee <[email protected]>
---
 groovy/src/main/java/org/apache/zeppelin/groovy/GroovyInterpreter.java | 1 -
 1 file changed, 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 f2a88ccf39..fd0a60cd80 100644
--- a/groovy/src/main/java/org/apache/zeppelin/groovy/GroovyInterpreter.java
+++ b/groovy/src/main/java/org/apache/zeppelin/groovy/GroovyInterpreter.java
@@ -196,7 +196,6 @@ public class GroovyInterpreter extends Interpreter {
       if (object instanceof Thread) {
         try {
           Thread t = (Thread) object;
-          t.dumpStack();
           t.interrupt();
           //t.stop(); //TODO(dlukyanov): need some way to terminate maybe 
through GObject..
         } catch (Throwable t) {

Reply via email to