[GitHub] hit-lacus opened a new pull request #368: KYLIN-3187 Forbid calls to JDK APIs that use the default locale

2018-11-30 Thread GitBox
hit-lacus opened a new pull request #368: KYLIN-3187 Forbid calls to JDK APIs that use the default locale URL: https://github.com/apache/kylin/pull/368 Fix encode bugs. This is an automated message from the Apache Git

[GitHub] hit-lacus commented on issue #368: KYLIN-3187 Forbid calls to JDK APIs that use the default locale

2018-11-30 Thread GitBox
hit-lacus commented on issue #368: KYLIN-3187 Forbid calls to JDK APIs that use the default locale URL: https://github.com/apache/kylin/pull/368#issuecomment-443176896 ```sql select uid, act_type, count(*) as cnt from useraction as ua where act_type > '播放' group by uid,

[GitHub] woyumen4597 commented on issue #366: KYLIN-3707 add configuration for setting isolation-level for sqoop

2018-11-29 Thread GitBox
woyumen4597 commented on issue #366: KYLIN-3707 add configuration for setting isolation-level for sqoop URL: https://github.com/apache/kylin/pull/366#issuecomment-442820920 CI has passed.

[GitHub] shaofengshi commented on a change in pull request #348: added release notes, modified kylin and coprocessor upgrade guides

2018-12-04 Thread GitBox
shaofengshi commented on a change in pull request #348: added release notes, modified kylin and coprocessor upgrade guides URL: https://github.com/apache/kylin/pull/348#discussion_r238899445 ## File path: website/_docs/index.cn.md ## @@ -9,21 +9,31 @@ permalink:

[GitHub] shaofengshi commented on a change in pull request #348: added release notes, modified kylin and coprocessor upgrade guides

2018-12-04 Thread GitBox
shaofengshi commented on a change in pull request #348: added release notes, modified kylin and coprocessor upgrade guides URL: https://github.com/apache/kylin/pull/348#discussion_r238899442 ## File path: website/_docs/index.cn.md ## @@ -9,21 +9,31 @@ permalink:

[GitHub] shaofengshi commented on a change in pull request #348: added release notes, modified kylin and coprocessor upgrade guides

2018-12-04 Thread GitBox
shaofengshi commented on a change in pull request #348: added release notes, modified kylin and coprocessor upgrade guides URL: https://github.com/apache/kylin/pull/348#discussion_r238899806 ## File path: website/download/index.cn.md ## @@ -21,7 +21,7 @@ title: 下载

[GitHub] shaofengshi commented on a change in pull request #348: added release notes, modified kylin and coprocessor upgrade guides

2018-12-04 Thread GitBox
shaofengshi commented on a change in pull request #348: added release notes, modified kylin and coprocessor upgrade guides URL: https://github.com/apache/kylin/pull/348#discussion_r238900055 ## File path: website/download/index.md ## @@ -20,7 +20,7 @@ You can verify the

[GitHub] shaofengshi commented on a change in pull request #348: added release notes, modified kylin and coprocessor upgrade guides

2018-12-04 Thread GitBox
shaofengshi commented on a change in pull request #348: added release notes, modified kylin and coprocessor upgrade guides URL: https://github.com/apache/kylin/pull/348#discussion_r238900904 ## File path: website/_docs/howto/howto_upgrade.md ## @@ -1,112 +1,146 @@ ---

[GitHub] shaofengshi commented on a change in pull request #348: added release notes, modified kylin and coprocessor upgrade guides

2018-12-04 Thread GitBox
shaofengshi commented on a change in pull request #348: added release notes, modified kylin and coprocessor upgrade guides URL: https://github.com/apache/kylin/pull/348#discussion_r238900295 ## File path: website/_data/docs.yml ## @@ -78,6 +77,10 @@ -

[GitHub] shaofengshi commented on a change in pull request #348: added release notes, modified kylin and coprocessor upgrade guides

2018-12-04 Thread GitBox
shaofengshi commented on a change in pull request #348: added release notes, modified kylin and coprocessor upgrade guides URL: https://github.com/apache/kylin/pull/348#discussion_r238900533 ## File path: website/_docs/howto/howto_upgrade.md ## @@ -1,112 +1,146 @@ ---

[GitHub] asfgit commented on issue #371: KYLIN-3695, fix lose decimal scale value in column type decimal(a, b)

2018-12-04 Thread GitBox
asfgit commented on issue #371: KYLIN-3695, fix lose decimal scale value in column type decimal(a, b) URL: https://github.com/apache/kylin/pull/371#issuecomment-444363598 Can one of the admins verify this patch? This is an

[GitHub] Emiya0306 opened a new pull request #371: KYLIN-3695, fix lose decimal scale value in column type decimal(a, b)

2018-12-04 Thread GitBox
Emiya0306 opened a new pull request #371: KYLIN-3695, fix lose decimal scale value in column type decimal(a, b) URL: https://github.com/apache/kylin/pull/371 This is an automated message from the Apache Git Service. To

[GitHub] luguosheng1314 closed pull request #371: KYLIN-3695, fix lose decimal scale value in column type decimal(a, b)

2018-12-04 Thread GitBox
luguosheng1314 closed pull request #371: KYLIN-3695, fix lose decimal scale value in column type decimal(a, b) URL: https://github.com/apache/kylin/pull/371 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

[GitHub] codecov-io commented on issue #371: KYLIN-3695, fix lose decimal scale value in column type decimal(a, b)

2018-12-04 Thread GitBox
codecov-io commented on issue #371: KYLIN-3695, fix lose decimal scale value in column type decimal(a, b) URL: https://github.com/apache/kylin/pull/371#issuecomment-444367864 # [Codecov](https://codecov.io/gh/apache/kylin/pull/371?src=pr=h1) Report > Merging

[GitHub] codecov-io commented on issue #372: KYLIN-1111 Ignore unsupported hive column types when sync hive table

2018-12-05 Thread GitBox
codecov-io commented on issue #372: KYLIN- Ignore unsupported hive column types when sync hive table URL: https://github.com/apache/kylin/pull/372#issuecomment-444610689 # [Codecov](https://codecov.io/gh/apache/kylin/pull/372?src=pr=h1) Report > Merging

[GitHub] asfgit commented on issue #372: KYLIN-1111 Ignore unsupported hive column types when sync hive table

2018-12-05 Thread GitBox
asfgit commented on issue #372: KYLIN- Ignore unsupported hive column types when sync hive table URL: https://github.com/apache/kylin/pull/372#issuecomment-444600629 Can one of the admins verify this patch? This is an

[GitHub] whuwb opened a new pull request #372: KYLIN-1111 Ignore unsupported hive column types when sync hive table

2018-12-05 Thread GitBox
whuwb opened a new pull request #372: KYLIN- Ignore unsupported hive column types when sync hive table URL: https://github.com/apache/kylin/pull/372 Issue : https://issues.apache.org/jira/browse/KYLIN- Solution : Filter in HiveMetadataExplorer#extractColumnFromMeta How to

[GitHub] GinaZhai opened a new pull request #359: KYLIN-3628 The wrong result when a query with one lookup table

2018-11-23 Thread GitBox
GinaZhai opened a new pull request #359: KYLIN-3628 The wrong result when a query with one lookup table URL: https://github.com/apache/kylin/pull/359 This is an automated message from the Apache Git Service. To respond to

[GitHub] asfgit commented on issue #359: KYLIN-3628 The wrong result when a query with one lookup table

2018-11-23 Thread GitBox
asfgit commented on issue #359: KYLIN-3628 The wrong result when a query with one lookup table URL: https://github.com/apache/kylin/pull/359#issuecomment-441250502 Can one of the admins verify this patch? This is an

[GitHub] whuwb opened a new pull request #358: KYLIN-3559 Use Splitter for splitting String

2018-11-22 Thread GitBox
whuwb opened a new pull request #358: KYLIN-3559 Use Splitter for splitting String URL: https://github.com/apache/kylin/pull/358 Issue : https://issues.apache.org/jira/browse/KYLIN-3559 Solution : 1. Leverage Splitter in Guava to create split/splitByComma and refactor splitAndTrim

[GitHub] asfgit commented on issue #358: KYLIN-3559 Use Splitter for splitting String

2018-11-22 Thread GitBox
asfgit commented on issue #358: KYLIN-3559 Use Splitter for splitting String URL: https://github.com/apache/kylin/pull/358#issuecomment-441171380 Can one of the admins verify this patch? This is an automated message from the

[GitHub] coveralls commented on issue #358: KYLIN-3559 Use Splitter for splitting String

2018-11-23 Thread GitBox
coveralls commented on issue #358: KYLIN-3559 Use Splitter for splitting String URL: https://github.com/apache/kylin/pull/358#issuecomment-441174625 ## Pull Request Test Coverage Report for [Build 3878](https://coveralls.io/builds/20255839) * **9** of **42** **(21.43%)** changed

[GitHub] codecov-io commented on issue #358: KYLIN-3559 Use Splitter for splitting String

2018-11-23 Thread GitBox
codecov-io commented on issue #358: KYLIN-3559 Use Splitter for splitting String URL: https://github.com/apache/kylin/pull/358#issuecomment-441174344 # [Codecov](https://codecov.io/gh/apache/kylin/pull/358?src=pr=h1) Report > Merging

[GitHub] Aaaaaaron commented on a change in pull request #352: KYLIN-3669, add log to indicate the case when using "GTStreamAggregateScanner".

2018-11-20 Thread GitBox
Aaron commented on a change in pull request #352: KYLIN-3669, add log to indicate the case when using "GTStreamAggregateScanner". URL: https://github.com/apache/kylin/pull/352#discussion_r235229952 ## File path:

[GitHub] Aaaaaaron commented on a change in pull request #352: KYLIN-3669, add log to indicate the case when using "GTStreamAggregateScanner".

2018-11-20 Thread GitBox
Aaron commented on a change in pull request #352: KYLIN-3669, add log to indicate the case when using "GTStreamAggregateScanner". URL: https://github.com/apache/kylin/pull/352#discussion_r235229952 ## File path:

[GitHub] shaofengshi commented on a change in pull request #352: KYLIN-3669, add log to indicate the case when using "GTStreamAggregateScanner".

2018-11-20 Thread GitBox
shaofengshi commented on a change in pull request #352: KYLIN-3669, add log to indicate the case when using "GTStreamAggregateScanner". URL: https://github.com/apache/kylin/pull/352#discussion_r235217694 ## File path:

[GitHub] codecov-io commented on issue #347: KYLIN-3666 HDFS metadata url not be recognized

2018-11-20 Thread GitBox
codecov-io commented on issue #347: KYLIN-3666 HDFS metadata url not be recognized URL: https://github.com/apache/kylin/pull/347#issuecomment-440520921 # [Codecov](https://codecov.io/gh/apache/kylin/pull/347?src=pr=h1) Report > Merging

[GitHub] coveralls edited a comment on issue #347: KYLIN-3666 HDFS metadata url not be recognized

2018-11-20 Thread GitBox
coveralls edited a comment on issue #347: KYLIN-3666 HDFS metadata url not be recognized URL: https://github.com/apache/kylin/pull/347#issuecomment-439015589 ## Pull Request Test Coverage Report for [Build 3870](https://coveralls.io/builds/20216347) * **0** of **4** **(0.0%)**

[GitHub] shaofengshi closed pull request #352: KYLIN-3669, add log to indicate the case when using "GTStreamAggregateScanner".

2018-11-20 Thread GitBox
shaofengshi closed pull request #352: KYLIN-3669, add log to indicate the case when using "GTStreamAggregateScanner". URL: https://github.com/apache/kylin/pull/352 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake

[GitHub] shaofengshi commented on a change in pull request #352: KYLIN-3669, add log to indicate the case when using "GTStreamAggregateScanner".

2018-11-20 Thread GitBox
shaofengshi commented on a change in pull request #352: KYLIN-3669, add log to indicate the case when using "GTStreamAggregateScanner". URL: https://github.com/apache/kylin/pull/352#discussion_r235249257 ## File path:

[GitHub] shaofengshi closed pull request #354: KYLIN-3697 minor, only kylin.sh start check-env

2018-11-20 Thread GitBox
shaofengshi closed pull request #354: KYLIN-3697 minor, only kylin.sh start check-env URL: https://github.com/apache/kylin/pull/354 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 closed pull request #350: Fix issue : KYLIN-3631

2018-11-19 Thread GitBox
shaofengshi closed pull request #350: Fix issue : KYLIN-3631 URL: https://github.com/apache/kylin/pull/350 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

[GitHub] coveralls edited a comment on issue #350: Fix issue : KYLIN-3631

2018-11-19 Thread GitBox
coveralls edited a comment on issue #350: Fix issue : KYLIN-3631 URL: https://github.com/apache/kylin/pull/350#issuecomment-439873914 ## Pull Request Test Coverage Report for [Build 3860](https://coveralls.io/builds/20193220) * **1** of **2** **(50.0%)** changed or added relevant

[GitHub] codecov-io commented on issue #351: KYLIN-3290 Leverage getDecalredConstructor().newInstance() instead of newInstance()

2018-11-19 Thread GitBox
codecov-io commented on issue #351: KYLIN-3290 Leverage getDecalredConstructor().newInstance() instead of newInstance() URL: https://github.com/apache/kylin/pull/351#issuecomment-440175653 # [Codecov](https://codecov.io/gh/apache/kylin/pull/351?src=pr=h1) Report > :exclamation: No

[GitHub] shaofengshi closed pull request #351: KYLIN-3290 Leverage getDecalredConstructor().newInstance() instead of newInstance()

2018-11-20 Thread GitBox
shaofengshi closed pull request #351: KYLIN-3290 Leverage getDecalredConstructor().newInstance() instead of newInstance() URL: https://github.com/apache/kylin/pull/351 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[GitHub] whuwb opened a new pull request #351: KYLIN-3290 Leverage getDecalredConstructor().newInstance() instead of newInstance()

2018-11-19 Thread GitBox
whuwb opened a new pull request #351: KYLIN-3290 Leverage getDecalredConstructor().newInstance() instead of newInstance() URL: https://github.com/apache/kylin/pull/351 Issue : https://issues.apache.org/jira/browse/KYLIN-3290 How to fix : Leverage getDecalredConstructor().newInstance()

[GitHub] asfgit commented on issue #351: KYLIN-3290 Leverage getDecalredConstructor().newInstance() instead of newInstance()

2018-11-19 Thread GitBox
asfgit commented on issue #351: KYLIN-3290 Leverage getDecalredConstructor().newInstance() instead of newInstance() URL: https://github.com/apache/kylin/pull/351#issuecomment-440170481 Can one of the admins verify this patch?

[GitHub] shaofengshi closed pull request #347: KYLIN-3666 HDFS metadata url not be recognized

2018-11-21 Thread GitBox
shaofengshi closed pull request #347: KYLIN-3666 HDFS metadata url not be recognized URL: https://github.com/apache/kylin/pull/347 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] asfgit commented on issue #355: detailed configurations could be override in the Cube-level

2018-11-21 Thread GitBox
asfgit commented on issue #355: detailed configurations could be override in the Cube-level URL: https://github.com/apache/kylin/pull/355#issuecomment-440662329 Can one of the admins verify this patch? This is an automated

[GitHub] DingqianZhou opened a new pull request #355: detailed configurations could be override in the Cube-level

2018-11-21 Thread GitBox
DingqianZhou opened a new pull request #355: detailed configurations could be override in the Cube-level URL: https://github.com/apache/kylin/pull/355 This is an automated message from the Apache Git Service. To respond to

[GitHub] coveralls commented on issue #360: [KYLIN-3665]-Partition time column may never be added fixed.

2018-11-26 Thread GitBox
coveralls commented on issue #360: [KYLIN-3665]-Partition time column may never be added fixed. URL: https://github.com/apache/kylin/pull/360#issuecomment-441565049 ## Pull Request Test Coverage Report for [Build 3883](https://coveralls.io/builds/20286362) * **10** of **14**

[GitHub] shaofengshi closed pull request #357: KYLIN-3694 Bug fix and revert guava version

2018-11-26 Thread GitBox
shaofengshi closed pull request #357: KYLIN-3694 Bug fix and revert guava version URL: https://github.com/apache/kylin/pull/357 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] FrancisLiang opened a new pull request #360: [KYLIN-3665]-Partition time column may never be added fixed.

2018-11-26 Thread GitBox
FrancisLiang opened a new pull request #360: [KYLIN-3665]-Partition time column may never be added fixed. URL: https://github.com/apache/kylin/pull/360 Fixed [KYLIN-3665]: Partition time column may never be added. This is an

[GitHub] asfgit commented on issue #360: [KYLIN-3665]-Partition time column may never be added fixed.

2018-11-26 Thread GitBox
asfgit commented on issue #360: [KYLIN-3665]-Partition time column may never be added fixed. URL: https://github.com/apache/kylin/pull/360#issuecomment-441558684 Can one of the admins verify this patch? This is an automated

[GitHub] shaofengshi closed pull request #358: KYLIN-3559 Use Splitter for splitting String

2018-11-26 Thread GitBox
shaofengshi closed pull request #358: KYLIN-3559 Use Splitter for splitting String URL: https://github.com/apache/kylin/pull/358 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 closed pull request #344: #KYLIN-3684, HIVE_LIB is not set or not resolved correctly

2018-11-26 Thread GitBox
shaofengshi closed pull request #344: #KYLIN-3684, HIVE_LIB is not set or not resolved correctly URL: https://github.com/apache/kylin/pull/344 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

[GitHub] shaofengshi commented on issue #349: KYLIN-3694 Kylin On Druid Storage

2018-11-26 Thread GitBox
shaofengshi commented on issue #349: KYLIN-3694 Kylin On Druid Storage URL: https://github.com/apache/kylin/pull/349#issuecomment-441596761 This is not needed now, let's use the "kylin_on_druid" branch to stage the changes.

[GitHub] vaful opened a new pull request #361: KYLIN-3704 upgrade calcite version to 1.16.0-kylin-r2 version

2018-11-26 Thread GitBox
vaful opened a new pull request #361: KYLIN-3704 upgrade calcite version to 1.16.0-kylin-r2 version URL: https://github.com/apache/kylin/pull/361 This is an automated message from the Apache Git Service. To respond to the

[GitHub] asfgit commented on issue #361: KYLIN-3704 upgrade calcite version to 1.16.0-kylin-r2 version

2018-11-26 Thread GitBox
asfgit commented on issue #361: KYLIN-3704 upgrade calcite version to 1.16.0-kylin-r2 version URL: https://github.com/apache/kylin/pull/361#issuecomment-441600947 Can one of the admins verify this patch? This is an automated

[GitHub] shaofengshi closed pull request #363: KYLIN-3597 fix static code issues

2018-11-27 Thread GitBox
shaofengshi closed pull request #363: KYLIN-3597 fix static code issues URL: https://github.com/apache/kylin/pull/363 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

[GitHub] shaofengshi closed pull request #364: KYLIN-3559 Use Splitter for splitting String

2018-11-27 Thread GitBox
shaofengshi closed pull request #364: KYLIN-3559 Use Splitter for splitting String URL: https://github.com/apache/kylin/pull/364 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] coveralls commented on issue #362: KYLIN-3700 Quote sql identities when creating flat table

2018-11-27 Thread GitBox
coveralls commented on issue #362: KYLIN-3700 Quote sql identities when creating flat table URL: https://github.com/apache/kylin/pull/362#issuecomment-442330731 ## Pull Request Test Coverage Report for [Build 3896](https://coveralls.io/builds/20329464) * **72** of **244**

[GitHub] codecov-io edited a comment on issue #362: KYLIN-3700 Quote sql identities when creating flat table

2018-11-27 Thread GitBox
codecov-io edited a comment on issue #362: KYLIN-3700 Quote sql identities when creating flat table URL: https://github.com/apache/kylin/pull/362#issuecomment-441728994 # [Codecov](https://codecov.io/gh/apache/kylin/pull/362?src=pr=h1) Report > Merging

[GitHub] coveralls edited a comment on issue #362: KYLIN-3700 Quote sql identities when creating flat table

2018-11-27 Thread GitBox
coveralls edited a comment on issue #362: KYLIN-3700 Quote sql identities when creating flat table URL: https://github.com/apache/kylin/pull/362#issuecomment-442330731 ## Pull Request Test Coverage Report for [Build 3897](https://coveralls.io/builds/20329685) * **72** of **244**

[GitHub] codecov-io edited a comment on issue #362: KYLIN-3700 Quote sql identities when creating flat table

2018-11-27 Thread GitBox
codecov-io edited a comment on issue #362: KYLIN-3700 Quote sql identities when creating flat table URL: https://github.com/apache/kylin/pull/362#issuecomment-441728994 # [Codecov](https://codecov.io/gh/apache/kylin/pull/362?src=pr=h1) Report > Merging

[GitHub] Aaaaaaron commented on a change in pull request #365: KYLIN-2578, clean deprecated config("kylin.job.lock") usage in the code.

2018-11-27 Thread GitBox
Aaron commented on a change in pull request #365: KYLIN-2578, clean deprecated config("kylin.job.lock") usage in the code. URL: https://github.com/apache/kylin/pull/365#discussion_r236961076 ## File path:

[GitHub] asfgit commented on issue #365: KYLIN-2578, clean deprecated config("kylin.job.lock") usage in the code.

2018-11-27 Thread GitBox
asfgit commented on issue #365: KYLIN-2578, clean deprecated config("kylin.job.lock") usage in the code. URL: https://github.com/apache/kylin/pull/365#issuecomment-442342755 Can one of the admins verify this patch? This is

[GitHub] Aaaaaaron opened a new pull request #365: KYLIN-2578, clean deprecated config("kylin.job.lock") usage in the code.

2018-11-27 Thread GitBox
Aaron opened a new pull request #365: KYLIN-2578, clean deprecated config("kylin.job.lock") usage in the code. URL: https://github.com/apache/kylin/pull/365 This is an automated message from the Apache Git Service. To

[GitHub] shaofengshi closed pull request #362: KYLIN-3700 Quote sql identities when creating flat table

2018-11-27 Thread GitBox
shaofengshi closed pull request #362: KYLIN-3700 Quote sql identities when creating flat table URL: https://github.com/apache/kylin/pull/362 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

[GitHub] asfgit commented on issue #356: KYLIN-3665 Partition time column may never be used

2018-11-22 Thread GitBox
asfgit commented on issue #356: KYLIN-3665 Partition time column may never be used URL: https://github.com/apache/kylin/pull/356#issuecomment-441061435 Can one of the admins verify this patch? This is an automated message

[GitHub] Wayne1c opened a new pull request #356: KYLIN-3665 Partition time column may never be used

2018-11-22 Thread GitBox
Wayne1c opened a new pull request #356: KYLIN-3665 Partition time column may never be used URL: https://github.com/apache/kylin/pull/356 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] coveralls commented on issue #356: KYLIN-3665 Partition time column may never be used

2018-11-22 Thread GitBox
coveralls commented on issue #356: KYLIN-3665 Partition time column may never be used URL: https://github.com/apache/kylin/pull/356#issuecomment-441068851 ## Pull Request Test Coverage Report for [Build 3872](https://coveralls.io/builds/20247518) * **37** of **38** **(97.37%)**

[GitHub] codecov-io edited a comment on issue #357: KYLIN-3694 Bug fix and revert guava version

2018-11-22 Thread GitBox
codecov-io edited a comment on issue #357: KYLIN-3694 Bug fix and revert guava version URL: https://github.com/apache/kylin/pull/357#issuecomment-441145690 # [Codecov](https://codecov.io/gh/apache/kylin/pull/357?src=pr=h1) Report > :exclamation: No coverage uploaded for pull request

[GitHub] codecov-io commented on issue #356: KYLIN-3665 Partition time column may never be used

2018-11-22 Thread GitBox
codecov-io commented on issue #356: KYLIN-3665 Partition time column may never be used URL: https://github.com/apache/kylin/pull/356#issuecomment-441150667 # [Codecov](https://codecov.io/gh/apache/kylin/pull/356?src=pr=h1) Report > Merging

[GitHub] asfgit commented on issue #357: KYLIN-3694 Bug fix and revert guava version

2018-11-22 Thread GitBox
asfgit commented on issue #357: KYLIN-3694 Bug fix and revert guava version URL: https://github.com/apache/kylin/pull/357#issuecomment-441142155 Can one of the admins verify this patch? This is an automated message from the

[GitHub] hit-lacus opened a new pull request #357: KYLIN-3694 Bug fix and revert guava version

2018-11-22 Thread GitBox
hit-lacus opened a new pull request #357: KYLIN-3694 Bug fix and revert guava version URL: https://github.com/apache/kylin/pull/357 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] codecov-io commented on issue #357: KYLIN-3694 Bug fix and revert guava version

2018-11-22 Thread GitBox
codecov-io commented on issue #357: KYLIN-3694 Bug fix and revert guava version URL: https://github.com/apache/kylin/pull/357#issuecomment-441145690 # [Codecov](https://codecov.io/gh/apache/kylin/pull/357?src=pr=h1) Report > :exclamation: No coverage uploaded for pull request base

[GitHub] asfgit commented on issue #353: KYLIN-3697 Check port availability when starts kylin instance

2018-11-20 Thread GitBox
asfgit commented on issue #353: KYLIN-3697 Check port availability when starts kylin instance URL: https://github.com/apache/kylin/pull/353#issuecomment-440273296 Can one of the admins verify this patch? This is an automated

[GitHub] DingqianZhou opened a new pull request #353: KYLIN-3697 Check port availability when starts kylin instance

2018-11-20 Thread GitBox
DingqianZhou opened a new pull request #353: KYLIN-3697 Check port availability when starts kylin instance URL: https://github.com/apache/kylin/pull/353 This is an automated message from the Apache Git Service. To respond

[GitHub] shaofengshi closed pull request #353: KYLIN-3697 Check port availability when starts kylin instance

2018-11-20 Thread GitBox
shaofengshi closed pull request #353: KYLIN-3697 Check port availability when starts kylin instance URL: https://github.com/apache/kylin/pull/353 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:

[GitHub] Aaaaaaron opened a new pull request #352: KYLIN-3669, add log to indicate the case when using "GTStreamAggregateScanner".

2018-11-20 Thread GitBox
Aaron opened a new pull request #352: KYLIN-3669, add log to indicate the case when using "GTStreamAggregateScanner". URL: https://github.com/apache/kylin/pull/352 …Scanner. This is an automated message from the Apache

[GitHub] codecov-io commented on issue #353: KYLIN-3697 Check port availability when starts kylin instance

2018-11-20 Thread GitBox
codecov-io commented on issue #353: KYLIN-3697 Check port availability when starts kylin instance URL: https://github.com/apache/kylin/pull/353#issuecomment-440281351 # [Codecov](https://codecov.io/gh/apache/kylin/pull/353?src=pr=h1) Report > Merging

[GitHub] asfgit commented on issue #352: KYLIN-3669, add log to indicate the case when using "GTStreamAggregateScanner".

2018-11-20 Thread GitBox
asfgit commented on issue #352: KYLIN-3669, add log to indicate the case when using "GTStreamAggregateScanner". URL: https://github.com/apache/kylin/pull/352#issuecomment-440223647 Can one of the admins verify this patch?

[GitHub] asfgit commented on issue #349: KYLIN-3694 Kylin On Druid Storage

2018-11-19 Thread GitBox
asfgit commented on issue #349: KYLIN-3694 Kylin On Druid Storage URL: https://github.com/apache/kylin/pull/349#issuecomment-439816566 Can one of the admins verify this patch? This is an automated message from the Apache Git

[GitHub] hit-lacus opened a new pull request #349: KYLIN-3694 Kylin On Druid Storage

2018-11-19 Thread GitBox
hit-lacus opened a new pull request #349: KYLIN-3694 Kylin On Druid Storage URL: https://github.com/apache/kylin/pull/349 **Meituan** Kylin team has implemented a new storage engine for Kylin: Druid Storage Engine This is

[GitHub] coveralls commented on issue #349: KYLIN-3694 Kylin On Druid Storage

2018-11-19 Thread GitBox
coveralls commented on issue #349: KYLIN-3694 Kylin On Druid Storage URL: https://github.com/apache/kylin/pull/349#issuecomment-439823023 ## Pull Request Test Coverage Report for [Build 3857](https://coveralls.io/builds/20175448) * **3** of **2886** **(0.1%)** changed or added

[GitHub] whuwb opened a new pull request #350: Fix issue : KYLIN-3631

2018-11-19 Thread GitBox
whuwb opened a new pull request #350: Fix issue : KYLIN-3631 URL: https://github.com/apache/kylin/pull/350 Issue : https://issues.apache.org/jira/browse/KYLIN-3631 How to fix : 1. Find all the Arrays.sort and replace with Arrays.parallelSort and add comments. 2. Run UT &

[GitHub] asfgit commented on issue #350: Fix issue : KYLIN-3631

2018-11-19 Thread GitBox
asfgit commented on issue #350: Fix issue : KYLIN-3631 URL: https://github.com/apache/kylin/pull/350#issuecomment-439868082 Can one of the admins verify this patch? This is an automated message from the Apache Git Service. To

[GitHub] coveralls commented on issue #350: Fix issue : KYLIN-3631

2018-11-19 Thread GitBox
coveralls commented on issue #350: Fix issue : KYLIN-3631 URL: https://github.com/apache/kylin/pull/350#issuecomment-439873914 ## Pull Request Test Coverage Report for [Build 3858](https://coveralls.io/builds/20178446) * **1** of **2** **(50.0%)** changed or added relevant lines

[GitHub] codecov-io commented on issue #350: Fix issue : KYLIN-3631

2018-11-19 Thread GitBox
codecov-io commented on issue #350: Fix issue : KYLIN-3631 URL: https://github.com/apache/kylin/pull/350#issuecomment-439874742 # [Codecov](https://codecov.io/gh/apache/kylin/pull/350?src=pr=h1) Report > Merging [#350](https://codecov.io/gh/apache/kylin/pull/350?src=pr=desc) into

[GitHub] shaofengshi commented on a change in pull request #350: Fix issue : KYLIN-3631

2018-11-19 Thread GitBox
shaofengshi commented on a change in pull request #350: Fix issue : KYLIN-3631 URL: https://github.com/apache/kylin/pull/350#discussion_r234635124 ## File path: core-cube/src/test/java/org/apache/kylin/cube/cuboid/CuboidSchedulerTest.java ## @@ -118,8 +118,13 @@ private

[GitHub] whuwb commented on a change in pull request #350: Fix issue : KYLIN-3631

2018-11-19 Thread GitBox
whuwb commented on a change in pull request #350: Fix issue : KYLIN-3631 URL: https://github.com/apache/kylin/pull/350#discussion_r234681139 ## File path: core-cube/src/test/java/org/apache/kylin/cube/cuboid/CuboidSchedulerTest.java ## @@ -118,8 +118,13 @@ private void

[GitHub] codecov-io commented on issue #354: KYLIN-3697 minor, only kylin.sh start check-env

2018-11-20 Thread GitBox
codecov-io commented on issue #354: KYLIN-3697 minor, only kylin.sh start check-env URL: https://github.com/apache/kylin/pull/354#issuecomment-440300354 # [Codecov](https://codecov.io/gh/apache/kylin/pull/354?src=pr=h1) Report > Merging

[GitHub] asfgit commented on issue #354: minor, only kylin.sh start check-env

2018-11-20 Thread GitBox
asfgit commented on issue #354: minor, only kylin.sh start check-env URL: https://github.com/apache/kylin/pull/354#issuecomment-440292695 Can one of the admins verify this patch? This is an automated message from the Apache

[GitHub] DingqianZhou opened a new pull request #354: minor, only kylin.sh start check-env

2018-11-20 Thread GitBox
DingqianZhou opened a new pull request #354: minor, only kylin.sh start check-env URL: https://github.com/apache/kylin/pull/354 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] Wayne1c closed pull request #356: KYLIN-3665 Partition time column may never be used

2018-11-26 Thread GitBox
Wayne1c closed pull request #356: KYLIN-3665 Partition time column may never be used URL: https://github.com/apache/kylin/pull/356 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] hit-lacus commented on issue #362: KYLIN-3700 Quote sql identities when creating flat table

2018-11-26 Thread GitBox
hit-lacus commented on issue #362: KYLIN-3700 Quote sql identities when creating flat table URL: https://github.com/apache/kylin/pull/362#issuecomment-441872165 Local CI pass. This is an automated message from the Apache Git

[GitHub] shaofengshi commented on issue #355: detailed configurations could be override in the Cube-level

2018-11-27 Thread GitBox
shaofengshi commented on issue #355: detailed configurations could be override in the Cube-level URL: https://github.com/apache/kylin/pull/355#issuecomment-441965061 Merged with patch, thank you Dingqian! This is an

[GitHub] shaofengshi closed pull request #355: detailed configurations could be override in the Cube-level

2018-11-27 Thread GitBox
shaofengshi closed pull request #355: detailed configurations could be override in the Cube-level URL: https://github.com/apache/kylin/pull/355 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

[GitHub] shaofengshi closed pull request #361: KYLIN-3704 upgrade calcite version to 1.16.0-kylin-r2 version

2018-11-26 Thread GitBox
shaofengshi closed pull request #361: KYLIN-3704 upgrade calcite version to 1.16.0-kylin-r2 version URL: https://github.com/apache/kylin/pull/361 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:

[GitHub] FrancisLiang commented on issue #360: KYLIN-3665 Partition time column may never be added

2018-11-27 Thread GitBox
FrancisLiang commented on issue #360: KYLIN-3665 Partition time column may never be added URL: https://github.com/apache/kylin/pull/360#issuecomment-441988794 > Hi Francis, the change looks good. While the commit message didn't comply with Apache's style. Could you please update the

[GitHub] shaofengshi opened a new pull request #363: KYLIN-3597 fix static code issues

2018-11-27 Thread GitBox
shaofengshi opened a new pull request #363: KYLIN-3597 fix static code issues URL: https://github.com/apache/kylin/pull/363 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] asfgit commented on issue #363: KYLIN-3597 fix static code issues

2018-11-27 Thread GitBox
asfgit commented on issue #363: KYLIN-3597 fix static code issues URL: https://github.com/apache/kylin/pull/363#issuecomment-442011938 Can one of the admins verify this patch? This is an automated message from the Apache Git

[GitHub] shaofengshi commented on issue #360: [KYLIN-3665]-Partition time column may never be added fixed.

2018-11-27 Thread GitBox
shaofengshi commented on issue #360: [KYLIN-3665]-Partition time column may never be added fixed. URL: https://github.com/apache/kylin/pull/360#issuecomment-441987516 Hi Francis, the change looks good. While the commit message didn't comply with Apache's style. Could you please update the

[GitHub] shaofengshi closed pull request #360: KYLIN-3665 Partition time column may never be added

2018-11-27 Thread GitBox
shaofengshi closed pull request #360: KYLIN-3665 Partition time column may never be added URL: https://github.com/apache/kylin/pull/360 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

[GitHub] asfgit commented on issue #364: KYLIN-3559 Use Splitter for splitting String

2018-11-27 Thread GitBox
asfgit commented on issue #364: KYLIN-3559 Use Splitter for splitting String URL: https://github.com/apache/kylin/pull/364#issuecomment-442076913 Can one of the admins verify this patch? This is an automated message from the

[GitHub] whuwb opened a new pull request #364: KYLIN-3559 Use Splitter for splitting String

2018-11-27 Thread GitBox
whuwb opened a new pull request #364: KYLIN-3559 Use Splitter for splitting String URL: https://github.com/apache/kylin/pull/364 Issue: https://issues.apache.org/jira/browse/KYLIN-3559 Solution: Use Splitter for splitting String IT: [INFO] Reactor Summary for Apache

[GitHub] codecov-io commented on issue #363: KYLIN-3597 fix static code issues

2018-11-27 Thread GitBox
codecov-io commented on issue #363: KYLIN-3597 fix static code issues URL: https://github.com/apache/kylin/pull/363#issuecomment-442078521 # [Codecov](https://codecov.io/gh/apache/kylin/pull/363?src=pr=h1) Report > :exclamation: No coverage uploaded for pull request base

[GitHub] woyumen4597 commented on issue #366: KYLIN-3707 add configuration for setting isolation-level for sqoop

2018-11-28 Thread GitBox
woyumen4597 commented on issue #366: KYLIN-3707 add configuration for setting isolation-level for sqoop URL: https://github.com/apache/kylin/pull/366#issuecomment-442461461 @shaofengshi Good suggestion,but I think each datasource has its own configuration,including adaptor and xml

[GitHub] coveralls edited a comment on issue #350: Fix issue : KYLIN-3631

2018-11-19 Thread GitBox
coveralls edited a comment on issue #350: Fix issue : KYLIN-3631 URL: https://github.com/apache/kylin/pull/350#issuecomment-439873914 ## Pull Request Test Coverage Report for [Build 3859](https://coveralls.io/builds/20183226) * **1** of **2** **(50.0%)** changed or added relevant

<    2   3   4   5   6   7   8   9   10   11   >