[ 
https://issues.apache.org/jira/browse/HIVE-23088?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17068527#comment-17068527
 ] 

Panagiotis Garefalakis commented on HIVE-23088:
-----------------------------------------------

Hey [~volodymyr] thanks for reporting this!
The method used in a couple of other places as well e.g.: HiveServer2, 
LlapWrappedAppender 
It would make sense to replace it all together.

> Using Strings from log4j breaks non-log4j users
> -----------------------------------------------
>
>                 Key: HIVE-23088
>                 URL: https://issues.apache.org/jira/browse/HIVE-23088
>             Project: Hive
>          Issue Type: Bug
>    Affects Versions: 3.1.2
>            Reporter: Vova Vysotskyi
>            Priority: Major
>
> {{HookUtils}} uses explicitly {{org.apache.logging.log4j.util.Strings}} class 
> from log4j, but it may break clients who use other loggers and should exclude 
> log4j from the classpath.
> {{commons-lang}} has class {{StringUtils}} which may be used as a replacement 
> for this one:
>  {{Strings.isBlank}} -> {{StringUtils.isBlank}}.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to