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); }