kylin-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Shao Feng Shi (Jira)" <>
Subject [jira] [Updated] (KYLIN-4132) Kylin needn't use "" to import api-util.jar
Date Mon, 02 Mar 2020 14:39:00 GMT


Shao Feng Shi updated KYLIN-4132:
    Fix Version/s: v3.0.2

> Kylin needn't use "" to import api-util.jar
> --------------------------------------------------------------------------------
>                 Key: KYLIN-4132
>                 URL:
>             Project: Kylin
>          Issue Type: Improvement
>          Components: REST Service
>         Environment: Fusion Insight
>            Reporter: zhao jintao
>            Assignee: wangrupeng
>            Priority: Major
>             Fix For: v3.0.2
>   Original Estimate: 168h
>  Remaining Estimate: 168h
> Hi Team:
> I read kylin code find that JobService class and ProjectService class import "" 
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 "" to "org.apache.commons.lang.StringUtils.isEmpty(String)"
to avoid this problem.
> Best wishes
> Yours   zjt.

This message was sent by Atlassian Jira

View raw message