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 <[email protected]>
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 <[email protected]>
(cherry picked from commit 8d73bfc50c5746913b762a5d83acf6d9628efc10)
Signed-off-by: Philipp Dallig <[email protected]>
---
.../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);
}