[GitHub] shaofengshi commented on issue #137: add the compatibility for new version of hbase

2018-05-04 Thread GitBox
shaofengshi commented on issue #137: add the compatibility for new version of hbase URL: https://github.com/apache/kylin/pull/137#issuecomment-386539310 Hello Wankunde, Could you please create a JIRA to describe this change? For example:

[GitHub] wankunde commented on issue #137: add the compatibility for new version of hbase

2018-05-04 Thread GitBox
wankunde commented on issue #137: add the compatibility for new version of hbase URL: https://github.com/apache/kylin/pull/137#issuecomment-386546266 Hi,shaofengshi, I just created a issue : https://issues.apache.org/jira/browse/KYLIN-3367

[GitHub] wankunde opened a new pull request #137: add the compatibility for new version of hbase

2018-05-04 Thread GitBox
wankunde opened a new pull request #137: add the compatibility for new version of hbase URL: https://github.com/apache/kylin/pull/137 Newly hbase version add some new method in `Table` and `ResultScanner` interface. So we should add the implementation method.

[GitHub] coveralls commented on issue #138: Apache kylin 3359

2018-05-07 Thread GitBox
coveralls commented on issue #138: Apache kylin 3359 URL: https://github.com/apache/kylin/pull/138#issuecomment-387103592 ## Pull Request Test Coverage Report for [Build 3130](https://coveralls.io/builds/16867502) * **331** of **1178** **(28.1%)** changed or added relevant lines in

[GitHub] kyotoYaho opened a new pull request #138: Apache kylin 3359

2018-05-07 Thread GitBox
kyotoYaho opened a new pull request #138: Apache kylin 3359 URL: https://github.com/apache/kylin/pull/138 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the

[GitHub] asfgit commented on issue #138: Apache kylin 3359

2018-05-07 Thread GitBox
asfgit commented on issue #138: Apache kylin 3359 URL: https://github.com/apache/kylin/pull/138#issuecomment-387097127 Can one of the admins verify this patch? This is an automated message from the Apache Git Service. To

[GitHub] codecov-io commented on issue #138: Apache kylin 3359

2018-05-07 Thread GitBox
codecov-io commented on issue #138: Apache kylin 3359 URL: https://github.com/apache/kylin/pull/138#issuecomment-387110165 # [Codecov](https://codecov.io/gh/apache/kylin/pull/138?src=pr=h1) Report > Merging [#138](https://codecov.io/gh/apache/kylin/pull/138?src=pr=desc) into

[GitHub] codecov-io commented on issue #138: Apache kylin 3359

2018-05-14 Thread GitBox
codecov-io commented on issue #138: Apache kylin 3359 URL: https://github.com/apache/kylin/pull/138#issuecomment-387110165 # [Codecov](https://codecov.io/gh/apache/kylin/pull/138?src=pr=h1) Report > Merging [#138](https://codecov.io/gh/apache/kylin/pull/138?src=pr=desc) into

[GitHub] coveralls commented on issue #138: Apache kylin 3359

2018-05-14 Thread GitBox
coveralls commented on issue #138: Apache kylin 3359 URL: https://github.com/apache/kylin/pull/138#issuecomment-387103592 ## Pull Request Test Coverage Report for [Build 3143](https://coveralls.io/builds/16965788) * **344** of **1342** **(25.63%)** changed or added relevant lines

[GitHub] codecov-io commented on issue #138: Apache kylin 3359

2018-05-14 Thread GitBox
codecov-io commented on issue #138: Apache kylin 3359 URL: https://github.com/apache/kylin/pull/138#issuecomment-387110165 # [Codecov](https://codecov.io/gh/apache/kylin/pull/138?src=pr=h1) Report > Merging [#138](https://codecov.io/gh/apache/kylin/pull/138?src=pr=desc) into

[GitHub] coveralls commented on issue #138: Apache kylin 3359

2018-05-14 Thread GitBox
coveralls commented on issue #138: Apache kylin 3359 URL: https://github.com/apache/kylin/pull/138#issuecomment-387103592 ## Pull Request Test Coverage Report for [Build 3144](https://coveralls.io/builds/16968755) * **338** of **1338** **(25.26%)** changed or added relevant lines

[GitHub] coveralls commented on issue #138: Apache kylin 3359

2018-05-14 Thread GitBox
coveralls commented on issue #138: Apache kylin 3359 URL: https://github.com/apache/kylin/pull/138#issuecomment-387103592 ## Pull Request Test Coverage Report for [Build 3145](https://coveralls.io/builds/16983558) * **338** of **1347** **(25.09%)** changed or added relevant lines

[GitHub] codecov-io commented on issue #138: Apache kylin 3359

2018-05-14 Thread GitBox
codecov-io commented on issue #138: Apache kylin 3359 URL: https://github.com/apache/kylin/pull/138#issuecomment-387110165 # [Codecov](https://codecov.io/gh/apache/kylin/pull/138?src=pr=h1) Report > Merging [#138](https://codecov.io/gh/apache/kylin/pull/138?src=pr=desc) into

[GitHub] coveralls commented on issue #139: Kylin 3221 Some improvements for lookup table

2018-05-10 Thread GitBox
coveralls commented on issue #139: Kylin 3221 Some improvements for lookup table URL: https://github.com/apache/kylin/pull/139#issuecomment-387961369 ## Pull Request Test Coverage Report for [Build 3135](https://coveralls.io/builds/16916462) * **482** of **1504** **(32.05%)**

[GitHub] codecov-io commented on issue #139: Kylin 3221 Some improvements for lookup table

2018-05-10 Thread GitBox
codecov-io commented on issue #139: Kylin 3221 Some improvements for lookup table URL: https://github.com/apache/kylin/pull/139#issuecomment-387961787 # [Codecov](https://codecov.io/gh/apache/kylin/pull/139?src=pr=h1) Report > Merging

[GitHub] allenma opened a new pull request #139: Kylin 3221 Some improvements for lookup table

2018-05-09 Thread GitBox
allenma opened a new pull request #139: Kylin 3221 Some improvements for lookup table URL: https://github.com/apache/kylin/pull/139 Detail design can be found in jira: https://issues.apache.org/jira/browse/KYLIN-3221

[GitHub] asfgit commented on issue #139: Kylin 3221 Some improvements for lookup table

2018-05-09 Thread GitBox
asfgit commented on issue #139: Kylin 3221 Some improvements for lookup table URL: https://github.com/apache/kylin/pull/139#issuecomment-387958970 Can one of the admins verify this patch? This is an automated message from the

[GitHub] coveralls commented on issue #138: Apache kylin 3359

2018-05-11 Thread GitBox
coveralls commented on issue #138: Apache kylin 3359 URL: https://github.com/apache/kylin/pull/138#issuecomment-387103592 ## Pull Request Test Coverage Report for [Build 3136](https://coveralls.io/builds/16934535) * **331** of **1232** **(26.87%)** changed or added relevant lines

[GitHub] codecov-io commented on issue #138: Apache kylin 3359

2018-05-11 Thread GitBox
codecov-io commented on issue #138: Apache kylin 3359 URL: https://github.com/apache/kylin/pull/138#issuecomment-387110165 # [Codecov](https://codecov.io/gh/apache/kylin/pull/138?src=pr=h1) Report > Merging [#138](https://codecov.io/gh/apache/kylin/pull/138?src=pr=desc) into

[GitHub] shaofengshi commented on a change in pull request #138: Apache kylin 3359

2018-05-20 Thread GitBox
shaofengshi commented on a change in pull request #138: Apache kylin 3359 URL: https://github.com/apache/kylin/pull/138#discussion_r189458330 ## File path: core-metadata/src/main/java/org/apache/kylin/metadata/model/SumDynamicFunctionDesc.java ## @@ -0,0 +1,77 @@ +/* + *

[GitHub] shaofengshi commented on a change in pull request #138: Apache kylin 3359

2018-05-20 Thread GitBox
shaofengshi commented on a change in pull request #138: Apache kylin 3359 URL: https://github.com/apache/kylin/pull/138#discussion_r189458401 ## File path: core-metadata/src/main/java/org/apache/kylin/metadata/expression/ExpressionVisitor.java ## @@ -0,0 +1,35 @@ +/* + *

[GitHub] codecov-io commented on issue #139: Kylin 3221 Some improvements for lookup table

2018-05-24 Thread GitBox
codecov-io commented on issue #139: Kylin 3221 Some improvements for lookup table URL: https://github.com/apache/kylin/pull/139#issuecomment-387961787 # [Codecov](https://codecov.io/gh/apache/kylin/pull/139?src=pr=h1) Report > Merging

[GitHub] kyotoYaho commented on a change in pull request #138: Apache kylin 3359

2018-05-24 Thread GitBox
kyotoYaho commented on a change in pull request #138: Apache kylin 3359 URL: https://github.com/apache/kylin/pull/138#discussion_r190493021 ## File path: core-metadata/src/main/java/org/apache/kylin/metadata/model/SumDynamicFunctionDesc.java ## @@ -0,0 +1,77 @@ +/* + *

[GitHub] coveralls commented on issue #138: Apache kylin 3359

2018-05-24 Thread GitBox
coveralls commented on issue #138: Apache kylin 3359 URL: https://github.com/apache/kylin/pull/138#issuecomment-387103592 ## Pull Request Test Coverage Report for [Build 3158](https://coveralls.io/builds/17142110) * **338** of **1350** **(25.04%)** changed or added relevant lines

[GitHub] coveralls commented on issue #139: Kylin 3221 Some improvements for lookup table

2018-05-24 Thread GitBox
coveralls commented on issue #139: Kylin 3221 Some improvements for lookup table URL: https://github.com/apache/kylin/pull/139#issuecomment-387961369 ## Pull Request Test Coverage Report for [Build 3157](https://coveralls.io/builds/17141818) * **482** of **1504** **(32.05%)**

[GitHub] codecov-io commented on issue #138: Apache kylin 3359

2018-05-24 Thread GitBox
codecov-io commented on issue #138: Apache kylin 3359 URL: https://github.com/apache/kylin/pull/138#issuecomment-387110165 # [Codecov](https://codecov.io/gh/apache/kylin/pull/138?src=pr=h1) Report > Merging [#138](https://codecov.io/gh/apache/kylin/pull/138?src=pr=desc) into

[GitHub] shaofengshi commented on issue #137: add the compatibility for new version of hbase

2018-05-24 Thread GitBox
shaofengshi commented on issue #137: add the compatibility for new version of hbase URL: https://github.com/apache/kylin/pull/137#issuecomment-391629359 Merged in Kylin master branch, so close this PR. This is an automated

[GitHub] kyotoYaho commented on a change in pull request #138: Apache kylin 3359

2018-05-24 Thread GitBox
kyotoYaho commented on a change in pull request #138: Apache kylin 3359 URL: https://github.com/apache/kylin/pull/138#discussion_r190493215 ## File path: core-metadata/src/main/java/org/apache/kylin/metadata/expression/ExpressionVisitor.java ## @@ -0,0 +1,35 @@ +/* + *

[GitHub] shaofengshi closed pull request #137: add the compatibility for new version of hbase

2018-05-24 Thread GitBox
shaofengshi closed pull request #137: add the compatibility for new version of hbase URL: https://github.com/apache/kylin/pull/137 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] codecov-io commented on issue #138: Apache kylin 3359

2018-05-24 Thread GitBox
codecov-io commented on issue #138: Apache kylin 3359 URL: https://github.com/apache/kylin/pull/138#issuecomment-387110165 # [Codecov](https://codecov.io/gh/apache/kylin/pull/138?src=pr=h1) Report > Merging [#138](https://codecov.io/gh/apache/kylin/pull/138?src=pr=desc) into

[GitHub] coveralls commented on issue #138: Apache kylin 3359

2018-05-24 Thread GitBox
coveralls commented on issue #138: Apache kylin 3359 URL: https://github.com/apache/kylin/pull/138#issuecomment-387103592 ## Pull Request Test Coverage Report for [Build 3159](https://coveralls.io/builds/17143383) * **338** of **1357** **(24.91%)** changed or added relevant lines

[GitHub] codecov-io commented on issue #139: Kylin 3221 Some improvements for lookup table

2018-05-24 Thread GitBox
codecov-io commented on issue #139: Kylin 3221 Some improvements for lookup table URL: https://github.com/apache/kylin/pull/139#issuecomment-387961787 # [Codecov](https://codecov.io/gh/apache/kylin/pull/139?src=pr=h1) Report > Merging

[GitHub] coveralls commented on issue #139: Kylin 3221 Some improvements for lookup table

2018-05-24 Thread GitBox
coveralls commented on issue #139: Kylin 3221 Some improvements for lookup table URL: https://github.com/apache/kylin/pull/139#issuecomment-387961369 ## Pull Request Test Coverage Report for [Build 3160](https://coveralls.io/builds/17143741) * **483** of **1509** **(32.01%)**

[GitHub] asfgit commented on issue #140: Batch commits sync

2018-05-26 Thread GitBox
asfgit commented on issue #140: Batch commits sync URL: https://github.com/apache/kylin/pull/140#issuecomment-392260879 Can one of the admins verify this patch? This is an automated message from the Apache Git Service. To

[GitHub] liyang-gmt8 opened a new pull request #140: Batch commits sync

2018-05-26 Thread GitBox
liyang-gmt8 opened a new pull request #140: Batch commits sync URL: https://github.com/apache/kylin/pull/140 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use

[GitHub] liyang-gmt8 commented on issue #140: Batch commits sync

2018-05-26 Thread GitBox
liyang-gmt8 commented on issue #140: Batch commits sync URL: https://github.com/apache/kylin/pull/140#issuecomment-392261013 CI has passed This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] asfgit commented on issue #149: KYLIN-3401 change zip compress tool from java.util.zip to Apache comm…

2018-06-06 Thread GitBox
asfgit commented on issue #149: KYLIN-3401 change zip compress tool from java.util.zip to Apache comm… URL: https://github.com/apache/kylin/pull/149#issuecomment-395026347 Can one of the admins verify this patch? This is an

[GitHub] codecov-io commented on issue #149: KYLIN-3401 change zip compress tool from java.util.zip to Apache comm…

2018-06-06 Thread GitBox
codecov-io commented on issue #149: KYLIN-3401 change zip compress tool from java.util.zip to Apache comm… URL: https://github.com/apache/kylin/pull/149#issuecomment-395031843 # [Codecov](https://codecov.io/gh/apache/kylin/pull/149?src=pr=h1) Report > Merging

[GitHub] Wayne1c opened a new pull request #149: KYLIN-3401 change zip compress tool from java.util.zip to Apache comm…

2018-06-06 Thread GitBox
Wayne1c opened a new pull request #149: KYLIN-3401 change zip compress tool from java.util.zip to Apache comm… URL: https://github.com/apache/kylin/pull/149 …ons-compress library This is an automated message from the Apache

[GitHub] coveralls commented on issue #149: KYLIN-3401 change zip compress tool from java.util.zip to Apache comm…

2018-06-06 Thread GitBox
coveralls commented on issue #149: KYLIN-3401 change zip compress tool from java.util.zip to Apache comm… URL: https://github.com/apache/kylin/pull/149#issuecomment-395033516 ## Pull Request Test Coverage Report for [Build 3202](https://coveralls.io/builds/17346966) * **0** of

[GitHub] codecov-io commented on issue #152: KYLIN-3398 Fix inaccurate arithmetic operations

2018-06-10 Thread GitBox
codecov-io commented on issue #152: KYLIN-3398 Fix inaccurate arithmetic operations URL: https://github.com/apache/kylin/pull/152#issuecomment-396025190 # [Codecov](https://codecov.io/gh/apache/kylin/pull/152?src=pr=h1) Report > Merging

[GitHub] coveralls commented on issue #151: KYLIN-3094 Upgrade zookeeper version from 3.4.8 to 3.4.12

2018-06-10 Thread GitBox
coveralls commented on issue #151: KYLIN-3094 Upgrade zookeeper version from 3.4.8 to 3.4.12 URL: https://github.com/apache/kylin/pull/151#issuecomment-396024008 ## Pull Request Test Coverage Report for [Build 3212](https://coveralls.io/builds/17411252) * **0** of **0** changed

[GitHub] shaofengshi closed pull request #150: minor, fix NoClassDefFoundError: org/apache/commons/compress/archiver…

2018-06-09 Thread GitBox
shaofengshi closed pull request #150: minor, fix NoClassDefFoundError: org/apache/commons/compress/archiver… URL: https://github.com/apache/kylin/pull/150 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 #151: KYLIN-3094 Upgrade zookeeper version from 3.4.8 to 3.4.12

2018-06-10 Thread GitBox
codecov-io commented on issue #151: KYLIN-3094 Upgrade zookeeper version from 3.4.8 to 3.4.12 URL: https://github.com/apache/kylin/pull/151#issuecomment-396023882 # [Codecov](https://codecov.io/gh/apache/kylin/pull/151?src=pr=h1) Report > Merging

[GitHub] coveralls commented on issue #152: KYLIN-3398 Fix inaccurate arithmetic operations

2018-06-10 Thread GitBox
coveralls commented on issue #152: KYLIN-3398 Fix inaccurate arithmetic operations URL: https://github.com/apache/kylin/pull/152#issuecomment-396025219 ## Pull Request Test Coverage Report for [Build 3215](https://coveralls.io/builds/17411321) * **0** of **2** **(0.0%)** changed

[GitHub] nichunen opened a new pull request #151: KYLIN-3094 Upgrade zookeeper version from 3.4.8 to 3.4.12

2018-06-09 Thread GitBox
nichunen opened a new pull request #151: KYLIN-3094 Upgrade zookeeper version from 3.4.8 to 3.4.12 URL: https://github.com/apache/kylin/pull/151 This is an automated message from the Apache Git Service. To respond to the

[GitHub] asfgit commented on issue #151: KYLIN-3094 Upgrade zookeeper version from 3.4.8 to 3.4.12

2018-06-09 Thread GitBox
asfgit commented on issue #151: KYLIN-3094 Upgrade zookeeper version from 3.4.8 to 3.4.12 URL: https://github.com/apache/kylin/pull/151#issuecomment-396023040 Can one of the admins verify this patch? This is an automated

[GitHub] nichunen opened a new pull request #152: KYLIN-3398 Fix inaccurate arithmetic operations

2018-06-10 Thread GitBox
nichunen opened a new pull request #152: KYLIN-3398 Fix inaccurate arithmetic operations URL: https://github.com/apache/kylin/pull/152 In LookupTableToHFileJob#calculateShardNum This is an automated message from the Apache

[GitHub] asfgit commented on issue #152: KYLIN-3398 Fix inaccurate arithmetic operations

2018-06-10 Thread GitBox
asfgit commented on issue #152: KYLIN-3398 Fix inaccurate arithmetic operations URL: https://github.com/apache/kylin/pull/152#issuecomment-396024485 Can one of the admins verify this patch? This is an automated message from

[GitHub] shaofengshi closed pull request #151: KYLIN-3094 Upgrade zookeeper version from 3.4.8 to 3.4.12

2018-06-10 Thread GitBox
shaofengshi closed pull request #151: KYLIN-3094 Upgrade zookeeper version from 3.4.8 to 3.4.12 URL: https://github.com/apache/kylin/pull/151 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 #152: KYLIN-3398 Fix inaccurate arithmetic operations

2018-06-10 Thread GitBox
shaofengshi closed pull request #152: KYLIN-3398 Fix inaccurate arithmetic operations URL: https://github.com/apache/kylin/pull/152 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 #150: minor, fix NoClassDefFoundError: org/apache/commons/compress/archiver…

2018-06-09 Thread GitBox
coveralls commented on issue #150: minor, fix NoClassDefFoundError: org/apache/commons/compress/archiver… URL: https://github.com/apache/kylin/pull/150#issuecomment-395933226 ## Pull Request Test Coverage Report for [Build 3209](https://coveralls.io/builds/17407497) * **0** of

[GitHub] codecov-io commented on issue #150: minor, fix NoClassDefFoundError: org/apache/commons/compress/archiver…

2018-06-09 Thread GitBox
codecov-io commented on issue #150: minor, fix NoClassDefFoundError: org/apache/commons/compress/archiver… URL: https://github.com/apache/kylin/pull/150#issuecomment-395932870 # [Codecov](https://codecov.io/gh/apache/kylin/pull/150?src=pr=h1) Report > Merging

[GitHub] asfgit commented on issue #153: KYLIN-1948, add test to cover the case.

2018-06-11 Thread GitBox
asfgit commented on issue #153: KYLIN-1948, add test to cover the case. URL: https://github.com/apache/kylin/pull/153#issuecomment-396160212 Can one of the admins verify this patch? This is an automated message from the

[GitHub] tttMelody opened a new pull request #153: KYLIN-1948, add test to cover the case.

2018-06-11 Thread GitBox
tttMelody opened a new pull request #153: KYLIN-1948, add test to cover the case. URL: https://github.com/apache/kylin/pull/153 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] codecov-io commented on issue #153: KYLIN-1948, add test to cover the case.

2018-06-11 Thread GitBox
codecov-io commented on issue #153: KYLIN-1948, add test to cover the case. URL: https://github.com/apache/kylin/pull/153#issuecomment-396165401 # [Codecov](https://codecov.io/gh/apache/kylin/pull/153?src=pr=h1) Report > :exclamation: No coverage uploaded for pull request base

[GitHub] shaofengshi closed pull request #153: KYLIN-1948, add test to cover the case.

2018-06-11 Thread GitBox
shaofengshi closed pull request #153: KYLIN-1948, add test to cover the case. URL: https://github.com/apache/kylin/pull/153 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

[GitHub] codecov-io commented on issue #154: KYLIN-3388 remove create redistribute flat hive table step if not set…

2018-06-12 Thread GitBox
codecov-io commented on issue #154: KYLIN-3388 remove create redistribute flat hive table step if not set… URL: https://github.com/apache/kylin/pull/154#issuecomment-396535440 # [Codecov](https://codecov.io/gh/apache/kylin/pull/154?src=pr=h1) Report > Merging

[GitHub] Wayne1c opened a new pull request #154: KYLIN-3388 remove create redistribute flat hive table step if not set…

2018-06-12 Thread GitBox
Wayne1c opened a new pull request #154: KYLIN-3388 remove create redistribute flat hive table step if not set… URL: https://github.com/apache/kylin/pull/154 … distribute column This is an automated message from the Apache

[GitHub] asfgit commented on issue #154: KYLIN-3388 remove create redistribute flat hive table step if not set…

2018-06-12 Thread GitBox
asfgit commented on issue #154: KYLIN-3388 remove create redistribute flat hive table step if not set… URL: https://github.com/apache/kylin/pull/154#issuecomment-396530155 Can one of the admins verify this patch? This is an

[GitHub] coveralls commented on issue #154: KYLIN-3388 remove create redistribute flat hive table step if not set…

2018-06-12 Thread GitBox
coveralls commented on issue #154: KYLIN-3388 remove create redistribute flat hive table step if not set… URL: https://github.com/apache/kylin/pull/154#issuecomment-396561500 ## Pull Request Test Coverage Report for [Build 3226](https://coveralls.io/builds/17443154) * **0** of

[GitHub] asfgit commented on issue #150: minor, fix NoClassDefFoundError: org/apache/commons/compress/archiver…

2018-06-08 Thread GitBox
asfgit commented on issue #150: minor, fix NoClassDefFoundError: org/apache/commons/compress/archiver… URL: https://github.com/apache/kylin/pull/150#issuecomment-395931914 Can one of the admins verify this patch? This is an

[GitHub] Wayne1c opened a new pull request #150: minor, fix NoClassDefFoundError: org/apache/commons/compress/archiver…

2018-06-08 Thread GitBox
Wayne1c opened a new pull request #150: minor, fix NoClassDefFoundError: org/apache/commons/compress/archiver… URL: https://github.com/apache/kylin/pull/150 …s/examples/Archiver This is an automated message from the Apache

[GitHub] shaofengshi closed pull request #154: KYLIN-3388 remove create redistribute flat hive table step if not set…

2018-06-12 Thread GitBox
shaofengshi closed pull request #154: KYLIN-3388 remove create redistribute flat hive table step if not set… URL: https://github.com/apache/kylin/pull/154 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 #150: minor, fix NoClassDefFoundError: org/apache/commons/compress/archiver…

2018-06-08 Thread GitBox
codecov-io commented on issue #150: minor, fix NoClassDefFoundError: org/apache/commons/compress/archiver… URL: https://github.com/apache/kylin/pull/150#issuecomment-395932870 # [Codecov](https://codecov.io/gh/apache/kylin/pull/150?src=pr=h1) Report > Merging

[GitHub] coveralls commented on issue #150: minor, fix NoClassDefFoundError: org/apache/commons/compress/archiver…

2018-06-08 Thread GitBox
coveralls commented on issue #150: minor, fix NoClassDefFoundError: org/apache/commons/compress/archiver… URL: https://github.com/apache/kylin/pull/150#issuecomment-395933226 ## Pull Request Test Coverage Report for [Build 3208](https://coveralls.io/builds/17403226) * **0** of

[GitHub] coveralls commented on issue #140: Batch commits sync

2018-05-26 Thread GitBox
coveralls commented on issue #140: Batch commits sync URL: https://github.com/apache/kylin/pull/140#issuecomment-392262349 ## Pull Request Test Coverage Report for [Build 3167](https://coveralls.io/builds/17179805) * **16** of **66** **(24.24%)** changed or added relevant lines

[GitHub] coveralls commented on issue #141: KYLIN-3345 KYLIN-3363 KYLIN-3380

2018-05-26 Thread GitBox
coveralls commented on issue #141: KYLIN-3345KYLIN-3363 KYLIN-3380 URL: https://github.com/apache/kylin/pull/141#issuecomment-392269378 ## Pull Request Test Coverage Report for [Build 3168](https://coveralls.io/builds/17180502) * **12** of **45** **(26.67%)** changed or

[GitHub] lidongsjtu opened a new pull request #141: KYLIN-3345 KYLIN-3363 KYLIN-3380

2018-05-26 Thread GitBox
lidongsjtu opened a new pull request #141: KYLIN-3345KYLIN-3363 KYLIN-3380 URL: https://github.com/apache/kylin/pull/141 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] codecov-io commented on issue #141: KYLIN-3345 KYLIN-3363 KYLIN-3380

2018-05-26 Thread GitBox
codecov-io commented on issue #141: KYLIN-3345KYLIN-3363 KYLIN-3380 URL: https://github.com/apache/kylin/pull/141#issuecomment-392268993 # [Codecov](https://codecov.io/gh/apache/kylin/pull/141?src=pr=h1) Report > Merging

[GitHub] asfgit commented on issue #141: KYLIN-3345 KYLIN-3363 KYLIN-3380

2018-05-26 Thread GitBox
asfgit commented on issue #141: KYLIN-3345KYLIN-3363 KYLIN-3380 URL: https://github.com/apache/kylin/pull/141#issuecomment-392267519 Can one of the admins verify this patch? This is an automated message from the

[GitHub] liyang-gmt8 commented on issue #140: Batch commits sync

2018-05-26 Thread GitBox
liyang-gmt8 commented on issue #140: Batch commits sync URL: https://github.com/apache/kylin/pull/140#issuecomment-392261013 CI has passed ``` [INFO] Apache Kylin - Tool SUCCESS [ 7.187 s] [INFO] Apache Kylin - Tool Assembly

[GitHub] yiming187 closed pull request #145: KYLIN-3295, clean unused class:SQLDigestUtil

2018-05-30 Thread GitBox
yiming187 closed pull request #145: KYLIN-3295, clean unused class:SQLDigestUtil URL: https://github.com/apache/kylin/pull/145 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

[GitHub] yiming187 commented on issue #145: KYLIN-3295, clean unused class:SQLDigestUtil

2018-05-30 Thread GitBox
yiming187 commented on issue #145: KYLIN-3295, clean unused class:SQLDigestUtil URL: https://github.com/apache/kylin/pull/145#issuecomment-393212364 LGTM This is an automated message from the Apache Git Service. To respond to

[GitHub] codecov-io commented on issue #139: Kylin 3221 Some improvements for lookup table

2018-05-26 Thread GitBox
codecov-io commented on issue #139: Kylin 3221 Some improvements for lookup table URL: https://github.com/apache/kylin/pull/139#issuecomment-387961787 # [Codecov](https://codecov.io/gh/apache/kylin/pull/139?src=pr=h1) Report > Merging

[GitHub] shaofengshi closed pull request #140: Batch commits sync

2018-05-27 Thread GitBox
shaofengshi closed pull request #140: Batch commits sync URL: https://github.com/apache/kylin/pull/140 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL

[GitHub] shaofengshi commented on issue #140: Batch commits sync

2018-05-27 Thread GitBox
shaofengshi commented on issue #140: Batch commits sync URL: https://github.com/apache/kylin/pull/140#issuecomment-392323459 Looks good, CI passed in my side. This is an automated message from the Apache Git Service. To

[GitHub] coveralls commented on issue #139: Kylin 3221 Some improvements for lookup table

2018-05-26 Thread GitBox
coveralls commented on issue #139: Kylin 3221 Some improvements for lookup table URL: https://github.com/apache/kylin/pull/139#issuecomment-387961369 ## Pull Request Test Coverage Report for [Build 3172](https://coveralls.io/builds/17183878) * **493** of **1512** **(32.61%)**

[GitHub] lidongsjtu closed pull request #141: KYLIN-3345 KYLIN-3363 KYLIN-3380

2018-05-26 Thread GitBox
lidongsjtu closed pull request #141: KYLIN-3345KYLIN-3363 KYLIN-3380 URL: https://github.com/apache/kylin/pull/141 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

[GitHub] Wayne1c opened a new pull request #143: KYLIN-3250 Upgrade jetty version to 9.3.22.v20171030

2018-05-28 Thread GitBox
Wayne1c opened a new pull request #143: KYLIN-3250 Upgrade jetty version to 9.3.22.v20171030 URL: https://github.com/apache/kylin/pull/143 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] asfgit commented on issue #143: KYLIN-3250 Upgrade jetty version to 9.3.22.v20171030

2018-05-28 Thread GitBox
asfgit commented on issue #143: KYLIN-3250 Upgrade jetty version to 9.3.22.v20171030 URL: https://github.com/apache/kylin/pull/143#issuecomment-392636140 Can one of the admins verify this patch? This is an automated message

[GitHub] asfgit commented on issue #144: KYLIN-3372 upgrade jackson databind version to 2.9.5

2018-05-28 Thread GitBox
asfgit commented on issue #144: KYLIN-3372 upgrade jackson databind version to 2.9.5 URL: https://github.com/apache/kylin/pull/144#issuecomment-392636284 Can one of the admins verify this patch? This is an automated message

[GitHub] Wayne1c opened a new pull request #144: KYLIN-3372 upgrade jackson databind version to 2.9.5

2018-05-28 Thread GitBox
Wayne1c opened a new pull request #144: KYLIN-3372 upgrade jackson databind version to 2.9.5 URL: https://github.com/apache/kylin/pull/144 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] shaofengshi commented on issue #143: KYLIN-3250 Upgrade jetty version to 9.3.22.v20171030

2018-05-28 Thread GitBox
shaofengshi commented on issue #143: KYLIN-3250 Upgrade jetty version to 9.3.22.v20171030 URL: https://github.com/apache/kylin/pull/143#issuecomment-392636333 LGTM, thanks Long chao! This is an automated message from the

[GitHub] shaofengshi closed pull request #143: KYLIN-3250 Upgrade jetty version to 9.3.22.v20171030

2018-05-28 Thread GitBox
shaofengshi closed pull request #143: KYLIN-3250 Upgrade jetty version to 9.3.22.v20171030 URL: https://github.com/apache/kylin/pull/143 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] coveralls commented on issue #143: KYLIN-3250 Upgrade jetty version to 9.3.22.v20171030

2018-05-28 Thread GitBox
coveralls commented on issue #143: KYLIN-3250 Upgrade jetty version to 9.3.22.v20171030 URL: https://github.com/apache/kylin/pull/143#issuecomment-392638177 ## Pull Request Test Coverage Report for [Build 3177](https://coveralls.io/builds/17204753) * **0** of **0** changed or

[GitHub] codecov-io commented on issue #143: KYLIN-3250 Upgrade jetty version to 9.3.22.v20171030

2018-05-28 Thread GitBox
codecov-io commented on issue #143: KYLIN-3250 Upgrade jetty version to 9.3.22.v20171030 URL: https://github.com/apache/kylin/pull/143#issuecomment-392638206 # [Codecov](https://codecov.io/gh/apache/kylin/pull/143?src=pr=h1) Report > Merging

[GitHub] coveralls commented on issue #144: KYLIN-3372 upgrade jackson databind version to 2.9.5

2018-05-28 Thread GitBox
coveralls commented on issue #144: KYLIN-3372 upgrade jackson databind version to 2.9.5 URL: https://github.com/apache/kylin/pull/144#issuecomment-392638708 ## Pull Request Test Coverage Report for [Build 3178](https://coveralls.io/builds/17204781) * **0** of **0** changed or

[GitHub] shaofengshi closed pull request #144: KYLIN-3372 upgrade jackson databind version to 2.9.5

2018-05-28 Thread GitBox
shaofengshi closed pull request #144: KYLIN-3372 upgrade jackson databind version to 2.9.5 URL: https://github.com/apache/kylin/pull/144 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] shaofengshi commented on issue #144: KYLIN-3372 upgrade jackson databind version to 2.9.5

2018-05-28 Thread GitBox
shaofengshi commented on issue #144: KYLIN-3372 upgrade jackson databind version to 2.9.5 URL: https://github.com/apache/kylin/pull/144#issuecomment-392639325 LGTM, thanks long chao! This is an automated message from the

[GitHub] codecov-io commented on issue #144: KYLIN-3372 upgrade jackson databind version to 2.9.5

2018-05-28 Thread GitBox
codecov-io commented on issue #144: KYLIN-3372 upgrade jackson databind version to 2.9.5 URL: https://github.com/apache/kylin/pull/144#issuecomment-392640151 # [Codecov](https://codecov.io/gh/apache/kylin/pull/144?src=pr=h1) Report > Merging

[GitHub] asfgit commented on issue #145: KYLIN-3295, clean unused class:SQLDigestUtil

2018-05-28 Thread GitBox
asfgit commented on issue #145: KYLIN-3295, clean unused class:SQLDigestUtil URL: https://github.com/apache/kylin/pull/145#issuecomment-392649913 Can one of the admins verify this patch? This is an automated message from the

[GitHub] tttMelody opened a new pull request #145: KYLIN-3295, clean unused class:SQLDigestUtil

2018-05-28 Thread GitBox
tttMelody opened a new pull request #145: KYLIN-3295, clean unused class:SQLDigestUtil URL: https://github.com/apache/kylin/pull/145 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] codecov-io commented on issue #145: KYLIN-3295, clean unused class:SQLDigestUtil

2018-05-28 Thread GitBox
codecov-io commented on issue #145: KYLIN-3295, clean unused class:SQLDigestUtil URL: https://github.com/apache/kylin/pull/145#issuecomment-392651611 # [Codecov](https://codecov.io/gh/apache/kylin/pull/145?src=pr=h1) Report > Merging

[GitHub] coveralls commented on issue #145: KYLIN-3295, clean unused class:SQLDigestUtil

2018-05-28 Thread GitBox
coveralls commented on issue #145: KYLIN-3295, clean unused class:SQLDigestUtil URL: https://github.com/apache/kylin/pull/145#issuecomment-392651792 ## Pull Request Test Coverage Report for [Build 3181](https://coveralls.io/builds/17205617) * **0** of **0** changed or added

[GitHub] tttMelody opened a new pull request #146: KYLIN-3299, fix when year less than 1000 in int data type and date encoding

2018-05-28 Thread GitBox
tttMelody opened a new pull request #146: KYLIN-3299, fix when year less than 1000 in int data type and date encoding URL: https://github.com/apache/kylin/pull/146 …code. This is an automated message from the Apache Git

[GitHub] asfgit commented on issue #146: KYLIN-3299, fix when year less than 1000 in int data type and date encoding

2018-05-28 Thread GitBox
asfgit commented on issue #146: KYLIN-3299, fix when year less than 1000 in int data type and date encoding URL: https://github.com/apache/kylin/pull/146#issuecomment-392651882 Can one of the admins verify this patch? This

[GitHub] shaofengshi closed pull request #128: KYLIN-3336

2018-05-30 Thread GitBox
shaofengshi closed pull request #128: KYLIN-3336 URL: https://github.com/apache/kylin/pull/128 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), the

[GitHub] shaofengshi commented on a change in pull request #147: KYLIN-3161 Enforce global dictionary for bitmap count distinct column…

2018-06-03 Thread GitBox
shaofengshi commented on a change in pull request #147: KYLIN-3161 Enforce global dictionary for bitmap count distinct column… URL: https://github.com/apache/kylin/pull/147#discussion_r192590806 ## File path:

[GitHub] shaofengshi closed pull request #147: KYLIN-3161 Enforce global dictionary for bitmap count distinct column…

2018-06-03 Thread GitBox
shaofengshi closed pull request #147: KYLIN-3161 Enforce global dictionary for bitmap count distinct column… URL: https://github.com/apache/kylin/pull/147 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

  1   2   3   4   5   6   7   8   9   10   >