[
https://issues.apache.org/jira/browse/KYLIN-4132?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17048535#comment-17048535
]
ASF GitHub Bot commented on KYLIN-4132:
---------------------------------------
RupengWang commented on pull request #1129: KYLIN-4132 Kylin needn't use
"org.apache.directory.api.util.Strings"
URL: https://github.com/apache/kylin/pull/1129
Kylin needn't use "org.apache.directory.api.util.Strings" to import extra
dependency
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
> Kylin needn't use "org.apache.directory.api.util.Strings" to import
> api-util.jar
> --------------------------------------------------------------------------------
>
> Key: KYLIN-4132
> URL: https://issues.apache.org/jira/browse/KYLIN-4132
> Project: Kylin
> Issue Type: Improvement
> Components: REST Service
> Environment: Fusion Insight
> Reporter: zhao jintao
> Assignee: wangrupeng
> Priority: Major
> Original Estimate: 168h
> Remaining Estimate: 168h
>
> Hi Team:
> I read kylin code find that JobService class and ProjectService class import
> "org.apache.directory.api.util.Strings" use the function of
> "Strings.isEmpty(String)" from "api-util.jar" to judge whether a string is
> empty.
> This function can be replaced by importing
> "org.apache.commons.lang.StringUtils" use "StringUtils.isEmpty(String)" from
> “common-lang.jar”.
> The "api-util.jar" was dependent by hbase-client. When kylin build&package,
> there isn't "api-util.jar" in kylin.war. When start kylin process, it will be
> imported from hbase client lib. But if hbase client doesn't dependency
> "api-util.jar", kylin running failed and will throw exception.
> In fact, I encountered this problem in the latest Fusion-Insignt version.
> Kylin start failed because there isn't "api-util.jar" in hbase client lib.
>
> So I suggest replace "org.apache.directory.api.util.Strings.isEmpty(String)"
> to "org.apache.commons.lang.StringUtils.isEmpty(String)" to avoid this
> problem.
> Best wishes
> Yours zjt.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
