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

pdallig pushed a commit to branch branch-0.12
in repository https://gitbox.apache.org/repos/asf/zeppelin.git


The following commit(s) were added to refs/heads/branch-0.12 by this push:
     new 80a216af07 [ZEPPELIN-6300] Replace deprecated StringUtils.equals to 
Strings.CS.equals
80a216af07 is described below

commit 80a216af07fefb2e32303a769ef96c8ef286b0d5
Author: eunhwa99 <68810660+eunhw...@users.noreply.github.com>
AuthorDate: Tue Aug 26 22:59:11 2025 +0900

    [ZEPPELIN-6300] Replace deprecated StringUtils.equals to Strings.CS.equals
    
    ### What is this PR for?
    Replaces deprecated `StringUtils.equals` with `Strings.CS.equals` in 
`NotebookServer` for null-safe comparison.
    
    ### What type of PR is it?
    Refactoring
    
    ### Todos
    * [ ] - Task
    
    ### What is the Jira issue?
    * [ZEPPELIN-6300](https://issues.apache.org/jira/browse/ZEPPELIN-6300)
    
    ### How should this be tested?
    * Run existing unit tests for `NotebookServer` to confirm behavior is 
unchanged.
    ### Screenshots (if appropriate)
    
    ### Questions:
    * Does the license files need to update? No
    * Is there breaking changes for older versions? No
    * Does this needs documentation? No
    
    Closes #5049 from eunhwa99/ZEPPELIN-6300.
    
    Signed-off-by: Philipp Dallig <philipp.dal...@gmail.com>
    (cherry picked from commit 8d73bfc50c5746913b762a5d83acf6d9628efc10)
    Signed-off-by: Philipp Dallig <philipp.dal...@gmail.com>
---
 .../src/main/java/org/apache/zeppelin/socket/NotebookServer.java     | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git 
a/zeppelin-server/src/main/java/org/apache/zeppelin/socket/NotebookServer.java 
b/zeppelin-server/src/main/java/org/apache/zeppelin/socket/NotebookServer.java
index fabdd8eea6..8edb096e3f 100644
--- 
a/zeppelin-server/src/main/java/org/apache/zeppelin/socket/NotebookServer.java
+++ 
b/zeppelin-server/src/main/java/org/apache/zeppelin/socket/NotebookServer.java
@@ -51,6 +51,7 @@ import jakarta.websocket.OnOpen;
 import jakarta.websocket.Session;
 import jakarta.websocket.server.ServerEndpoint;
 import org.apache.commons.lang3.StringUtils;
+import org.apache.commons.lang3.Strings;
 import org.apache.commons.lang3.exception.ExceptionUtils;
 import org.apache.thrift.TException;
 import org.apache.zeppelin.common.Message;
@@ -812,8 +813,8 @@ public class NotebookServer implements 
AngularObjectRegistryListener,
 
       List<AngularObject> angularObjects = 
note.getAngularObjects(interpreterGroup.getId());
       for (AngularObject ao : angularObjects) {
-        if (StringUtils.equals(ao.getNoteId(), note.getId())
-            && StringUtils.equals(ao.getParagraphId(), paragraph.getId())) {
+        if (Strings.CS.equals(ao.getNoteId(), note.getId())
+            && Strings.CS.equals(ao.getParagraphId(), paragraph.getId())) {
           pushAngularObjectToRemoteRegistry(ao.getNoteId(), 
ao.getParagraphId(),
               ao.getName(), ao.get(), registry, interpreterGroup.getId(), 
conn);
         }

Reply via email to