Hi, dong
    Your contribution will be welcomed.
    If you are interested, you can create a pull request in github.

Best Regard
Zhou QianHao





On 3/6/15, 3:56 PM, "dong wang" <[email protected]> wrote:

>1, in common/src/main/java/org/apache/kylin/common/KylinConfig.java
>    public long getJobStepTimeout() {
>        return Long.parseLong(getOptional("kylin.job.step.timeout",
>String.valueOf(2 * 60 * 60)));
>    }
>
>it seems that the timeout is fixed, for some cases, the job step may takes
>much more time than 2hrs, could we put this in some config file?
>
>
>2, in script/download-tomcat.sh
>since we compare STRING instead of NUMBER, thus, is it more reasonable to
>change
>from:
>if [ ! `md5sum apache-tomcat-7.0.59.tar.gz | awk '{print $1}'` -eq
>"ec570258976edf9a833cd88fd9220909" ]
>
>to
>
>if [ `md5sum apache-tomcat-7.0.59.tar.gz | awk '{print $1}'` !=
>"ec570258976edf9a833cd88fd9220909" ]

Reply via email to