eunhwa99 commented on PR #5049: URL: https://github.com/apache/zeppelin/pull/5049#issuecomment-3223544968
> Why don't you use the following method as suggested by JavaDoc? `[Strings.CS.equals(CharSequence, CharSequence)](https://commons.apache.org/proper/commons-lang/apidocs/org/apache/commons/lang3/Strings.html#equals(java.lang.CharSequence,java.lang.CharSequence)) Thank you for the suggestion. I initially replaced the deprecated `StringUtils.equals` with `Objects.equals` to use a JDK standard null-safe comparison and avoid relying on external libraries. However, I agree with your point and have updated it to use `Strings.CS.equals` as recommended by the JavaDoc, ensuring null-safe comparison while following the project's Apache Commons guidelines. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: reviews-unsubscr...@zeppelin.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
