[GitHub] shaofengshi closed pull request #242: KYLIN-3575, fix unclosed JDBC connection in DriverTest.

2018-09-24 Thread GitBox
shaofengshi closed pull request #242: KYLIN-3575, fix unclosed JDBC connection in DriverTest. URL: https://github.com/apache/kylin/pull/242 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this

Re: Kylin-Master-JDK-1.8 failures

2018-09-24 Thread ShaoFeng Shi
Hi Daniel, thank you for fixing it and updating us. We will take care of it. Daniel Pono Takamori 于2018年9月24日周一 上午6:23写道: > Hey Kylin folks, > When clearing disk space on H26 I found that your job was pinned to it > (I've since moved the label for the job to the generic 'ubuntu' > label). The

Re: [Announce] Apache Kylin 2.5.0 released

2018-09-24 Thread ShaoFeng Shi
Adding a blog post to introduce this release, we encourage you to upgrade and provide feedback in the mailing list: English: https://kylin.apache.org/blog/2018/09/20/release-v2.5.0/ Chinese: https://kylin.apache.org/cn/blog/2018/09/20/release-v2.5.0/ Thanks! Billy Liu 于2018年9月19日周三 下午9:23写道:

[GitHub] asfgit commented on issue #243: Update to execute the beeline to find CLASSPATH

2018-09-24 Thread GitBox
asfgit commented on issue #243: Update to execute the beeline to find CLASSPATH URL: https://github.com/apache/kylin/pull/243#issuecomment-423965743 Can one of the admins verify this patch? This is an automated message from

[GitHub] asfgit commented on issue #243: Update to execute the beeline to find CLASSPATH

2018-09-24 Thread GitBox
asfgit commented on issue #243: Update to execute the beeline to find CLASSPATH URL: https://github.com/apache/kylin/pull/243#issuecomment-423965744 Can one of the admins verify this patch? This is an automated message from

[GitHub] rtjarvis opened a new pull request #243: Update to execute the beeline to find CLASSPATH

2018-09-24 Thread GitBox
rtjarvis opened a new pull request #243: Update to execute the beeline to find CLASSPATH URL: https://github.com/apache/kylin/pull/243 The beeline "set" hangs because there is no terminating ; This is an automated message

regrading model construction apache kylin

2018-09-24 Thread vishnu vardhan g
what's the need of lookup table in creation of model in apache kylin. If there is onle one table do we need lookup table even then? Thank you. regrads Vishnu Vardhan G Software Intern Orzota

[GitHub] shaofengshi closed pull request #241: KYLIN-3546 Add column which belongs to root fact table in join relation but lost

2018-09-24 Thread GitBox
shaofengshi closed pull request #241: KYLIN-3546 Add column which belongs to root fact table in join relation but lost URL: https://github.com/apache/kylin/pull/241 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[jira] [Created] (KYLIN-3587) Null value is used in IllegalStateException message

2018-09-24 Thread Ted Yu (JIRA)
Ted Yu created KYLIN-3587: - Summary: Null value is used in IllegalStateException message Key: KYLIN-3587 URL: https://issues.apache.org/jira/browse/KYLIN-3587 Project: Kylin Issue Type: Bug

[GitHub] asfgit commented on issue #244: KYLIN-3582 String compare should use equals method

2018-09-24 Thread GitBox
asfgit commented on issue #244: KYLIN-3582 String compare should use equals method URL: https://github.com/apache/kylin/pull/244#issuecomment-424014579 Can one of the admins verify this patch? This is an automated message

[GitHub] hit-lacus opened a new pull request #244: KYLIN-3582 String compare should use equals method

2018-09-24 Thread GitBox
hit-lacus opened a new pull request #244: KYLIN-3582 String compare should use equals method URL: https://github.com/apache/kylin/pull/244 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] asfgit commented on issue #244: KYLIN-3582 String compare should use equals method

2018-09-24 Thread GitBox
asfgit commented on issue #244: KYLIN-3582 String compare should use equals method URL: https://github.com/apache/kylin/pull/244#issuecomment-424014580 Can one of the admins verify this patch? This is an automated message

[jira] [Created] (KYLIN-3588) Potentially duplicate put to RemoveBlackoutRealizationsRule#filters map

2018-09-24 Thread Ted Yu (JIRA)
Ted Yu created KYLIN-3588: - Summary: Potentially duplicate put to RemoveBlackoutRealizationsRule#filters map Key: KYLIN-3588 URL: https://issues.apache.org/jira/browse/KYLIN-3588 Project: Kylin

[jira] [Created] (KYLIN-3589) Different type is used to check presence in Collection in AclPermissionUtil

2018-09-24 Thread Ted Yu (JIRA)
Ted Yu created KYLIN-3589: - Summary: Different type is used to check presence in Collection in AclPermissionUtil Key: KYLIN-3589 URL: https://issues.apache.org/jira/browse/KYLIN-3589 Project: Kylin

[jira] [Created] (KYLIN-3590) Missing argument to exception message String in ModelController

2018-09-24 Thread Ted Yu (JIRA)
Ted Yu created KYLIN-3590: - Summary: Missing argument to exception message String in ModelController Key: KYLIN-3590 URL: https://issues.apache.org/jira/browse/KYLIN-3590 Project: Kylin Issue Type:

[GitHub] shaofengshi closed pull request #244: KYLIN-3582 String compare should use equals method

2018-09-24 Thread GitBox
shaofengshi closed pull request #244: KYLIN-3582 String compare should use equals method URL: https://github.com/apache/kylin/pull/244 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a

[GitHub] shaofengshi commented on issue #243: Update to execute the beeline to find CLASSPATH

2018-09-24 Thread GitBox
shaofengshi commented on issue #243: Update to execute the beeline to find CLASSPATH URL: https://github.com/apache/kylin/pull/243#issuecomment-424167617 Hello rtjarvis, could you please create a JIRA with the issue description, and then associate the commit with a message "KYLIN-

[GitHub] caolijun1166 opened a new pull request #245: KYLIN-3586

2018-09-24 Thread GitBox
caolijun1166 opened a new pull request #245: KYLIN-3586 URL: https://github.com/apache/kylin/pull/245 Solve KYLIN-3586: Use parseInt() to parse a primitive instead of to use boxing/unboxing This is an automated message from

[GitHub] asfgit commented on issue #245: KYLIN-3586

2018-09-24 Thread GitBox
asfgit commented on issue #245: KYLIN-3586 URL: https://github.com/apache/kylin/pull/245#issuecomment-424172929 Can one of the admins verify this patch? This is an automated message from the Apache Git Service. To respond to

[GitHub] asfgit commented on issue #245: KYLIN-3586

2018-09-24 Thread GitBox
asfgit commented on issue #245: KYLIN-3586 URL: https://github.com/apache/kylin/pull/245#issuecomment-424172928 Can one of the admins verify this patch? This is an automated message from the Apache Git Service. To respond to

[GitHub] asfgit commented on issue #247: Fix travis ci

2018-09-24 Thread GitBox
asfgit commented on issue #247: Fix travis ci URL: https://github.com/apache/kylin/pull/247#issuecomment-42418 Can one of the admins verify this patch? This is an automated message from the Apache Git Service. To respond

[GitHub] asfgit commented on issue #247: Fix travis ci

2018-09-24 Thread GitBox
asfgit commented on issue #247: Fix travis ci URL: https://github.com/apache/kylin/pull/247#issuecomment-424189998 Can one of the admins verify this patch? This is an automated message from the Apache Git Service. To respond

[GitHub] shaofengshi opened a new pull request #247: Fix travis ci

2018-09-24 Thread GitBox
shaofengshi opened a new pull request #247: Fix travis ci URL: https://github.com/apache/kylin/pull/247 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the

[GitHub] shaofengshi closed pull request #247: Fix travis ci

2018-09-24 Thread GitBox
shaofengshi closed pull request #247: Fix travis ci URL: https://github.com/apache/kylin/pull/247 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a fork),

[GitHub] codecov-io commented on issue #247: Fix travis ci

2018-09-24 Thread GitBox
codecov-io commented on issue #247: Fix travis ci URL: https://github.com/apache/kylin/pull/247#issuecomment-424193243 # [Codecov](https://codecov.io/gh/apache/kylin/pull/247?src=pr=h1) Report > :exclamation: No coverage uploaded for pull request base (`master@d8c2a28`). [Click here to

[GitHub] codecov-io edited a comment on issue #247: Fix travis ci

2018-09-24 Thread GitBox
codecov-io edited a comment on issue #247: Fix travis ci URL: https://github.com/apache/kylin/pull/247#issuecomment-424193243 # [Codecov](https://codecov.io/gh/apache/kylin/pull/247?src=pr=h1) Report > :exclamation: No coverage uploaded for pull request base (`master@d8c2a28`). [Click

[GitHub] asfgit commented on issue #246: KYLIN-3574 Close underlying resource of lookup table

2018-09-24 Thread GitBox
asfgit commented on issue #246: KYLIN-3574 Close underlying resource of lookup table URL: https://github.com/apache/kylin/pull/246#issuecomment-424175244 Can one of the admins verify this patch? This is an automated message

[GitHub] asfgit commented on issue #246: KYLIN-3574 Close underlying resource of lookup table

2018-09-24 Thread GitBox
asfgit commented on issue #246: KYLIN-3574 Close underlying resource of lookup table URL: https://github.com/apache/kylin/pull/246#issuecomment-424175245 Can one of the admins verify this patch? This is an automated message

[GitHub] hit-lacus opened a new pull request #246: KYLIN-3574 Close underlying resource of lookup table

2018-09-24 Thread GitBox
hit-lacus opened a new pull request #246: KYLIN-3574 Close underlying resource of lookup table URL: https://github.com/apache/kylin/pull/246 HTable should be closed. This is an automated message from the Apache Git Service.