kylin-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF subversion and git services (Jira)" <>
Subject [jira] [Commented] (KYLIN-4132) Kylin needn't use "" to import api-util.jar
Date Mon, 02 Mar 2020 01:41:00 GMT


ASF subversion and git services commented on KYLIN-4132:

Commit 9a86c3731f800bd9b2ac70234249f462f8016fc8 in kylin's branch refs/heads/3.0.x from rupengwang
[;h=9a86c37 ]

KYLIN-4132 Kylin needn't use "" to import extra dependency

> 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
>   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