kylin-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "zhao jintao (JIRA)" <>
Subject [jira] [Created] (KYLIN-4132) Kylin needn't use "" to import api-util.jar
Date Tue, 13 Aug 2019 06:07:00 GMT
zhao jintao created KYLIN-4132:

             Summary: Kylin needn't use "" to import
                 Key: KYLIN-4132
             Project: Kylin
          Issue Type: Improvement
          Components: REST Service
         Environment: Fusion Insight
            Reporter: zhao jintao

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