[GitHub] chenzhx closed pull request #162: KYLIN-3440 Couldn't merge if there is an empty segment

2018-07-03 Thread GitBox
chenzhx closed pull request #162: KYLIN-3440 Couldn't merge if there is an empty segment URL: https://github.com/apache/kylin/pull/162 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] tttMelody opened a new pull request #164: KYLIN-3439, make default kylin.web.timezone in KylinConfigBase the same with kylin-defaults.properties

2018-07-04 Thread GitBox
tttMelody opened a new pull request #164: KYLIN-3439, make default kylin.web.timezone in KylinConfigBase the same with kylin-defaults.properties URL: https://github.com/apache/kylin/pull/164 …me with kylin-defaults.properties

[GitHub] asfgit commented on issue #164: KYLIN-3439, make default kylin.web.timezone in KylinConfigBase the same with kylin-defaults.properties

2018-07-04 Thread GitBox
asfgit commented on issue #164: KYLIN-3439, make default kylin.web.timezone in KylinConfigBase the same with kylin-defaults.properties URL: https://github.com/apache/kylin/pull/164#issuecomment-402611570 Can one of the admins verify this patch? -

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

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

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

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

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

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

[GitHub] tttMelody commented on issue #164: KYLIN-3439, make default kylin.web.timezone in KylinConfigBase the same with kylin-defaults.properties

2018-07-07 Thread GitBox
tttMelody commented on issue #164: KYLIN-3439, make default kylin.web.timezone in KylinConfigBase the same with kylin-defaults.properties URL: https://github.com/apache/kylin/pull/164#issuecomment-403219285 Roger, I will change default to UTC, thank shaofeng! --

[GitHub] tttMelody edited a comment on issue #164: KYLIN-3439, make default kylin.web.timezone in KylinConfigBase the same with kylin-defaults.properties

2018-07-07 Thread GitBox
tttMelody edited a comment on issue #164: KYLIN-3439, make default kylin.web.timezone in KylinConfigBase the same with kylin-defaults.properties URL: https://github.com/apache/kylin/pull/164#issuecomment-403219285 Roger that, I will change default to UTC, thanks shaofeng! -

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

2018-07-08 Thread GitBox
shaofengshi closed pull request #165: Batch sync commits URL: https://github.com/apache/kylin/pull/165 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 #165: Batch sync commits

2018-07-08 Thread GitBox
shaofengshi commented on issue #165: Batch sync commits URL: https://github.com/apache/kylin/pull/165#issuecomment-403274562 LGTM, except some commits for the same JIRA wasn't squashed; I manually merged them and pushed in master branch.

[GitHub] shaofengshi closed pull request #159: KYLIN-3403 filter code system add one type of Date

2018-07-08 Thread GitBox
shaofengshi closed pull request #159: KYLIN-3403 filter code system add one type of Date URL: https://github.com/apache/kylin/pull/159 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 #159: KYLIN-3403 filter code system add one type of Date

2018-07-08 Thread GitBox
shaofengshi commented on issue #159: KYLIN-3403 filter code system add one type of Date URL: https://github.com/apache/kylin/pull/159#issuecomment-403275852 Yifan has fixed this; The pr is not needed; This is an automated me

[GitHub] shaofengshi commented on issue #23: Kylin es

2018-07-08 Thread GitBox
shaofengshi commented on issue #23: Kylin es URL: https://github.com/apache/kylin/pull/23#issuecomment-403275910 This PR has been out of date. And it didn't introduce how it be designed, implemented and how to compare with other storage engine like HBase. Close it now. ---

[GitHub] coveralls commented on issue #164: KYLIN-3439, make default kylin.web.timezone in KylinConfigBase the same with kylin-defaults.properties

2018-07-11 Thread GitBox
coveralls commented on issue #164: KYLIN-3439, make default kylin.web.timezone in KylinConfigBase the same with kylin-defaults.properties URL: https://github.com/apache/kylin/pull/164#issuecomment-404215437 ## Pull Request Test Coverage Report for [Build 3316](https://coveralls.io/builds/1

[GitHub] Wayne1c opened a new pull request #166: KYLIN-3457 Distribute by multi column if not set distribute column du…

2018-07-17 Thread GitBox
Wayne1c opened a new pull request #166: KYLIN-3457 Distribute by multi column if not set distribute column du… URL: https://github.com/apache/kylin/pull/166 …ring the redistribute step This is an automated message from the Ap

[GitHub] asfgit commented on issue #166: KYLIN-3457 Distribute by multi column if not set distribute column du…

2018-07-17 Thread GitBox
asfgit commented on issue #166: KYLIN-3457 Distribute by multi column if not set distribute column du… URL: https://github.com/apache/kylin/pull/166#issuecomment-405621222 Can one of the admins verify this patch? This is an a

[GitHub] asfgit commented on issue #167: Kylin 3186

2018-07-17 Thread GitBox
asfgit commented on issue #167: Kylin 3186 URL: https://github.com/apache/kylin/pull/167#issuecomment-405622499 Can one of the admins verify this patch? This is an automated message from the Apache Git Service. To respond to t

[GitHub] Wayne1c opened a new pull request #167: Kylin 3186

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

[GitHub] coveralls commented on issue #166: KYLIN-3457 Distribute by multi column if not set distribute column du…

2018-07-17 Thread GitBox
coveralls commented on issue #166: KYLIN-3457 Distribute by multi column if not set distribute column du… URL: https://github.com/apache/kylin/pull/166#issuecomment-405647004 ## Pull Request Test Coverage Report for [Build 3321](https://coveralls.io/builds/18026428) * **0** of **19*

[GitHub] coveralls commented on issue #167: Kylin 3186

2018-07-17 Thread GitBox
coveralls commented on issue #167: Kylin 3186 URL: https://github.com/apache/kylin/pull/167#issuecomment-405649921 ## Pull Request Test Coverage Report for [Build 3322](https://coveralls.io/builds/18026544) * **0** of **19** **(0.0%)** changed or added relevant lines in **2** fil

[GitHub] shaofengshi commented on a change in pull request #166: KYLIN-3457 Distribute by multi column if not set distribute column du…

2018-07-17 Thread GitBox
shaofengshi commented on a change in pull request #166: KYLIN-3457 Distribute by multi column if not set distribute column du… URL: https://github.com/apache/kylin/pull/166#discussion_r203236629 ## File path: core-job/src/main/java/org/apache/kylin/job/JoinedFlatTable.java

[GitHub] shaofengshi commented on a change in pull request #166: KYLIN-3457 Distribute by multi column if not set distribute column du…

2018-07-17 Thread GitBox
shaofengshi commented on a change in pull request #166: KYLIN-3457 Distribute by multi column if not set distribute column du… URL: https://github.com/apache/kylin/pull/166#discussion_r203237528 ## File path: source-hive/src/main/java/org/apache/kylin/source/hive/HiveSparkInput.jav

[GitHub] GinaZhai opened a new pull request #168: Kylin 3255 cannot save cube

2018-07-17 Thread GitBox
GinaZhai opened a new pull request #168: Kylin 3255 cannot save cube URL: https://github.com/apache/kylin/pull/168 modify the code for null condition This is an automated message from the Apache Git Service. To respond to the

[GitHub] asfgit commented on issue #168: Kylin 3255 cannot save cube

2018-07-17 Thread GitBox
asfgit commented on issue #168: Kylin 3255 cannot save cube URL: https://github.com/apache/kylin/pull/168#issuecomment-405806870 Can one of the admins verify this patch? This is an automated message from the Apache Git Service

[GitHub] coveralls commented on issue #168: Kylin 3255 cannot save cube

2018-07-18 Thread GitBox
coveralls commented on issue #168: Kylin 3255 cannot save cube URL: https://github.com/apache/kylin/pull/168#issuecomment-405854131 ## Pull Request Test Coverage Report for [Build 3323](https://coveralls.io/builds/18039485) * **0** of **4** **(0.0%)** changed or added relevant lin

[GitHub] coveralls edited a comment on issue #168: Kylin 3255 cannot save cube

2018-07-18 Thread GitBox
coveralls edited a comment on issue #168: Kylin 3255 cannot save cube URL: https://github.com/apache/kylin/pull/168#issuecomment-405854131 ## Pull Request Test Coverage Report for [Build 3325](https://coveralls.io/builds/18042179) * **0** of **4** **(0.0%)** changed or added relev

[GitHub] codecov-io commented on issue #166: KYLIN-3457 Distribute by multi column if not set distribute column du…

2018-07-18 Thread GitBox
codecov-io commented on issue #166: KYLIN-3457 Distribute by multi column if not set distribute column du… URL: https://github.com/apache/kylin/pull/166#issuecomment-405909335 # [Codecov](https://codecov.io/gh/apache/kylin/pull/166?src=pr&el=h1) Report > Merging [#166](https://codecov.io

[GitHub] coveralls edited a comment on issue #166: KYLIN-3457 Distribute by multi column if not set distribute column du…

2018-07-18 Thread GitBox
coveralls edited a comment on issue #166: KYLIN-3457 Distribute by multi column if not set distribute column du… URL: https://github.com/apache/kylin/pull/166#issuecomment-405647004 ## Pull Request Test Coverage Report for [Build 3327](https://coveralls.io/builds/18042784) * **0** o

[GitHub] coveralls edited a comment on issue #166: KYLIN-3457 Distribute by multi column if not set distribute column du…

2018-07-18 Thread GitBox
coveralls edited a comment on issue #166: KYLIN-3457 Distribute by multi column if not set distribute column du… URL: https://github.com/apache/kylin/pull/166#issuecomment-405647004 ## Pull Request Test Coverage Report for [Build 3328](https://coveralls.io/builds/18042821) * **0** o

[GitHub] Wayne1c commented on issue #166: KYLIN-3457 Distribute by multi column if not set distribute column

2018-07-18 Thread GitBox
Wayne1c commented on issue #166: KYLIN-3457 Distribute by multi column if not set distribute column URL: https://github.com/apache/kylin/pull/166#issuecomment-405916552 pr has been updated. This is an automated message from t

[GitHub] coveralls edited a comment on issue #166: KYLIN-3457 Distribute by multi column if not set distribute column

2018-07-18 Thread GitBox
coveralls edited a comment on issue #166: KYLIN-3457 Distribute by multi column if not set distribute column URL: https://github.com/apache/kylin/pull/166#issuecomment-405647004 ## Pull Request Test Coverage Report for [Build 3329](https://coveralls.io/builds/18043239) * **0** of **

[GitHub] shaofengshi closed pull request #168: Kylin 3255 cannot save cube

2018-07-18 Thread GitBox
shaofengshi closed pull request #168: Kylin 3255 cannot save cube URL: https://github.com/apache/kylin/pull/168 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 (fr

[GitHub] GinaZhai opened a new pull request #169: Kafka upgrade to 1.0.0

2018-07-18 Thread GitBox
GinaZhai opened a new pull request #169: Kafka upgrade to 1.0.0 URL: https://github.com/apache/kylin/pull/169 kafka upgrade kafka upgrade again This is an automated message from the Apache Git Service. To respond to the

[GitHub] asfgit commented on issue #169: Kafka upgrade to 1.0.0

2018-07-18 Thread GitBox
asfgit commented on issue #169: Kafka upgrade to 1.0.0 URL: https://github.com/apache/kylin/pull/169#issuecomment-406145296 Can one of the admins verify this patch? This is an automated message from the Apache Git Service. To

[GitHub] codecov-io edited a comment on issue #166: KYLIN-3457 Distribute by multi column if not set distribute column

2018-07-18 Thread GitBox
codecov-io edited a comment on issue #166: KYLIN-3457 Distribute by multi column if not set distribute column URL: https://github.com/apache/kylin/pull/166#issuecomment-405909335 # [Codecov](https://codecov.io/gh/apache/kylin/pull/166?src=pr&el=h1) Report > Merging [#166](https://codecov

[GitHub] coveralls commented on issue #169: Kafka upgrade to 1.0.0

2018-07-18 Thread GitBox
coveralls commented on issue #169: Kafka upgrade to 1.0.0 URL: https://github.com/apache/kylin/pull/169#issuecomment-406147896 ## Pull Request Test Coverage Report for [Build 3332](https://coveralls.io/builds/18058061) * **0** of **0** changed or added relevant lines in **0** files

[GitHub] coveralls edited a comment on issue #166: KYLIN-3457 Distribute by multi column if not set distribute column

2018-07-18 Thread GitBox
coveralls edited a comment on issue #166: KYLIN-3457 Distribute by multi column if not set distribute column URL: https://github.com/apache/kylin/pull/166#issuecomment-405647004 ## Pull Request Test Coverage Report for [Build 3331](https://coveralls.io/builds/18058060) * **0** of **

[GitHub] codecov-io commented on issue #169: Kafka upgrade to 1.0.0

2018-07-18 Thread GitBox
codecov-io commented on issue #169: Kafka upgrade to 1.0.0 URL: https://github.com/apache/kylin/pull/169#issuecomment-406150734 # [Codecov](https://codecov.io/gh/apache/kylin/pull/169?src=pr&el=h1) Report > Merging [#169](https://codecov.io/gh/apache/kylin/pull/169?src=pr&el=desc) into

[GitHub] shaofengshi commented on a change in pull request #166: KYLIN-3457 Distribute by multi column if not set distribute column

2018-07-19 Thread GitBox
shaofengshi commented on a change in pull request #166: KYLIN-3457 Distribute by multi column if not set distribute column URL: https://github.com/apache/kylin/pull/166#discussion_r203622944 ## File path: core-job/src/main/java/org/apache/kylin/job/JoinedFlatTable.java ##

[GitHub] shaofengshi closed pull request #167: Kylin 3186

2018-07-19 Thread GitBox
shaofengshi closed pull request #167: Kylin 3186 URL: https://github.com/apache/kylin/pull/167 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 d

[GitHub] shaofengshi closed pull request #169: Kafka upgrade to 1.0.0

2018-07-19 Thread GitBox
shaofengshi closed pull request #169: Kafka upgrade to 1.0.0 URL: https://github.com/apache/kylin/pull/169 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] shaofengshi closed pull request #166: KYLIN-3457 Distribute by multi column if not set distribute column

2018-07-19 Thread GitBox
shaofengshi closed pull request #166: KYLIN-3457 Distribute by multi column if not set distribute column URL: https://github.com/apache/kylin/pull/166 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] Wayne1c opened a new pull request #170: KYLIN-3457 use cube level config

2018-07-19 Thread GitBox
Wayne1c opened a new pull request #170: KYLIN-3457 use cube level config URL: https://github.com/apache/kylin/pull/170 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub

[GitHub] asfgit commented on issue #170: KYLIN-3457 use cube level config

2018-07-19 Thread GitBox
asfgit commented on issue #170: KYLIN-3457 use cube level config URL: https://github.com/apache/kylin/pull/170#issuecomment-406201162 Can one of the admins verify this patch? This is an automated message from the Apache Git Se

[GitHub] shaofengshi closed pull request #170: KYLIN-3457 use cube level config

2018-07-19 Thread GitBox
shaofengshi closed pull request #170: KYLIN-3457 use cube level config URL: https://github.com/apache/kylin/pull/170 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 reques

[GitHub] GinaZhai opened a new pull request #171: Superset document

2018-07-22 Thread GitBox
GinaZhai opened a new pull request #171: Superset document URL: https://github.com/apache/kylin/pull/171 including Chinese and English document This is an automated message from the Apache Git Service. To respond to the messag

[GitHub] asfgit commented on issue #171: Superset document

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

[GitHub] Wayne1c opened a new pull request #172: KYLIN-3453 improve cube size estimation for topn, count distinct

2018-07-23 Thread GitBox
Wayne1c opened a new pull request #172: KYLIN-3453 improve cube size estimation for topn,count distinct URL: https://github.com/apache/kylin/pull/172 This is an automated message from the Apache Git Service. To respond to th

[GitHub] asfgit commented on issue #172: KYLIN-3453 improve cube size estimation for topn, count distinct

2018-07-23 Thread GitBox
asfgit commented on issue #172: KYLIN-3453 improve cube size estimation for topn,count distinct URL: https://github.com/apache/kylin/pull/172#issuecomment-407251898 Can one of the admins verify this patch? This is an automate

[GitHub] asfgit commented on issue #173: KYLIN-3418 User interface for hybrid model - Frontend

2018-07-23 Thread GitBox
asfgit commented on issue #173: KYLIN-3418 User interface for hybrid model - Frontend URL: https://github.com/apache/kylin/pull/173#issuecomment-407269285 Can one of the admins verify this patch? This is an automated message

[GitHub] Emiya0306 opened a new pull request #173: KYLIN-3418 User interface for hybrid model - Frontend

2018-07-23 Thread GitBox
Emiya0306 opened a new pull request #173: KYLIN-3418 User interface for hybrid model - Frontend URL: https://github.com/apache/kylin/pull/173 This is an automated message from the Apache Git Service. To respond to the messag

[GitHub] coveralls commented on issue #173: KYLIN-3418 User interface for hybrid model - Frontend

2018-07-23 Thread GitBox
coveralls commented on issue #173: KYLIN-3418 User interface for hybrid model - Frontend URL: https://github.com/apache/kylin/pull/173#issuecomment-407273221 ## Pull Request Test Coverage Report for [Build 3342](https://coveralls.io/builds/18129057) * **0** of **0** changed or add

[GitHub] Wayne1c opened a new pull request #174: KYLIN-3418 adjust hybrid API

2018-07-24 Thread GitBox
Wayne1c opened a new pull request #174: KYLIN-3418 adjust hybrid API URL: https://github.com/apache/kylin/pull/174 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and

[GitHub] asfgit commented on issue #174: KYLIN-3418 adjust hybrid API

2018-07-24 Thread GitBox
asfgit commented on issue #174: KYLIN-3418 adjust hybrid API URL: https://github.com/apache/kylin/pull/174#issuecomment-407431944 Can one of the admins verify this patch? This is an automated message from the Apache Git Servic

[GitHub] shaofengshi commented on a change in pull request #174: KYLIN-3418 adjust hybrid API

2018-07-24 Thread GitBox
shaofengshi commented on a change in pull request #174: KYLIN-3418 adjust hybrid API URL: https://github.com/apache/kylin/pull/174#discussion_r204965852 ## File path: server-base/src/main/java/org/apache/kylin/rest/job/HybridCubeCLI.java ## @@ -105,6 +105,18 @@ protected

[GitHub] shaofengshi commented on a change in pull request #174: KYLIN-3418 adjust hybrid API

2018-07-24 Thread GitBox
shaofengshi commented on a change in pull request #174: KYLIN-3418 adjust hybrid API URL: https://github.com/apache/kylin/pull/174#discussion_r204966171 ## File path: server-base/src/main/java/org/apache/kylin/rest/service/HybridService.java ## @@ -43,8 +44,7 @@ @Aut

[GitHub] shaofengshi commented on a change in pull request #174: KYLIN-3418 adjust hybrid API

2018-07-24 Thread GitBox
shaofengshi commented on a change in pull request #174: KYLIN-3418 adjust hybrid API URL: https://github.com/apache/kylin/pull/174#discussion_r204969453 ## File path: server-base/src/main/java/org/apache/kylin/rest/service/HybridService.java ## @@ -66,8 +66,7 @@ public Hy

[GitHub] shaofengshi commented on a change in pull request #173: KYLIN-3418 User interface for hybrid model - Frontend

2018-07-24 Thread GitBox
shaofengshi commented on a change in pull request #173: KYLIN-3418 User interface for hybrid model - Frontend URL: https://github.com/apache/kylin/pull/173#discussion_r204969762 ## File path: webapp/app/js/model/hybridCubeManager.js ## @@ -0,0 +1,60 @@ +/* + * Licensed to

[GitHub] shaofengshi closed pull request #171: Superset document

2018-07-24 Thread GitBox
shaofengshi closed pull request #171: Superset document URL: https://github.com/apache/kylin/pull/171 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] coveralls edited a comment on issue #173: KYLIN-3418 User interface for hybrid model - Frontend

2018-07-24 Thread GitBox
coveralls edited a comment on issue #173: KYLIN-3418 User interface for hybrid model - Frontend URL: https://github.com/apache/kylin/pull/173#issuecomment-407273221 ## Pull Request Test Coverage Report for [Build 3346](https://coveralls.io/builds/18151257) * **0** of **0** changed

[GitHub] Emiya0306 opened a new pull request #175: KYLIN-3418 User interface for hybrid model - Frontend

2018-07-25 Thread GitBox
Emiya0306 opened a new pull request #175: KYLIN-3418 User interface for hybrid model - Frontend URL: https://github.com/apache/kylin/pull/175 This is an automated message from the Apache Git Service. To respond to the messag

[GitHub] asfgit commented on issue #175: KYLIN-3418 User interface for hybrid model - Frontend

2018-07-25 Thread GitBox
asfgit commented on issue #175: KYLIN-3418 User interface for hybrid model - Frontend URL: https://github.com/apache/kylin/pull/175#issuecomment-407655261 Can one of the admins verify this patch? This is an automated message

[GitHub] coveralls commented on issue #175: KYLIN-3418 User interface for hybrid model - Frontend

2018-07-25 Thread GitBox
coveralls commented on issue #175: KYLIN-3418 User interface for hybrid model - Frontend URL: https://github.com/apache/kylin/pull/175#issuecomment-407663956 ## Pull Request Test Coverage Report for [Build 3347](https://coveralls.io/builds/18151829) * **0** of **0** changed or add

[GitHub] coveralls edited a comment on issue #175: KYLIN-3418 User interface for hybrid model - Frontend

2018-07-25 Thread GitBox
coveralls edited a comment on issue #175: KYLIN-3418 User interface for hybrid model - Frontend URL: https://github.com/apache/kylin/pull/175#issuecomment-407663956 ## Pull Request Test Coverage Report for [Build 3349](https://coveralls.io/builds/18152307) * **0** of **0** changed

[GitHub] Wayne1c opened a new pull request #176: KYLIN-3259 when delete cube, remove it from hybrid

2018-07-25 Thread GitBox
Wayne1c opened a new pull request #176: KYLIN-3259 when delete cube, remove it from hybrid URL: https://github.com/apache/kylin/pull/176 This is an automated message from the Apache Git Service. To respond to the message, pl

[GitHub] asfgit commented on issue #176: KYLIN-3259 when delete cube, remove it from hybrid

2018-07-25 Thread GitBox
asfgit commented on issue #176: KYLIN-3259 when delete cube, remove it from hybrid URL: https://github.com/apache/kylin/pull/176#issuecomment-407790746 Can one of the admins verify this patch? This is an automated message fro

[GitHub] codecov-io commented on issue #176: KYLIN-3259 when delete cube, remove it from hybrid

2018-07-25 Thread GitBox
codecov-io commented on issue #176: KYLIN-3259 when delete cube, remove it from hybrid URL: https://github.com/apache/kylin/pull/176#issuecomment-407798828 # [Codecov](https://codecov.io/gh/apache/kylin/pull/176?src=pr&el=h1) Report > Merging [#176](https://codecov.io/gh/apache/kylin/pul

[GitHub] GinaZhai opened a new pull request #177: KYLIN-3414 Optimize the cleanup of project L2 cache

2018-07-25 Thread GitBox
GinaZhai opened a new pull request #177: KYLIN-3414 Optimize the cleanup of project L2 cache URL: https://github.com/apache/kylin/pull/177 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] asfgit commented on issue #177: KYLIN-3414 Optimize the cleanup of project L2 cache

2018-07-25 Thread GitBox
asfgit commented on issue #177: KYLIN-3414 Optimize the cleanup of project L2 cache URL: https://github.com/apache/kylin/pull/177#issuecomment-407948158 Can one of the admins verify this patch? This is an automated message fr

[GitHub] shaofengshi closed pull request #176: KYLIN-3259 when delete cube, remove it from hybrid

2018-07-25 Thread GitBox
shaofengshi closed pull request #176: KYLIN-3259 when delete cube, remove it from hybrid URL: https://github.com/apache/kylin/pull/176 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 #177: KYLIN-3414 Optimize the cleanup of project L2 cache

2018-07-25 Thread GitBox
shaofengshi closed pull request #177: KYLIN-3414 Optimize the cleanup of project L2 cache URL: https://github.com/apache/kylin/pull/177 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 #177: KYLIN-3414 Optimize the cleanup of project L2 cache

2018-07-25 Thread GitBox
coveralls commented on issue #177: KYLIN-3414 Optimize the cleanup of project L2 cache URL: https://github.com/apache/kylin/pull/177#issuecomment-407952518 ## Pull Request Test Coverage Report for [Build 3352](https://coveralls.io/builds/18169841) * **1** of **12** **(8.33%)** ch

[GitHub] coveralls edited a comment on issue #175: KYLIN-3418 User interface for hybrid model - Frontend

2018-07-25 Thread GitBox
coveralls edited a comment on issue #175: KYLIN-3418 User interface for hybrid model - Frontend URL: https://github.com/apache/kylin/pull/175#issuecomment-407663956 ## Pull Request Test Coverage Report for [Build 3355](https://coveralls.io/builds/18170010) * **0** of **0** changed

[GitHub] shaofengshi closed pull request #174: KYLIN-3418 Adjust hybrid API

2018-07-26 Thread GitBox
shaofengshi closed pull request #174: KYLIN-3418 Adjust hybrid API URL: https://github.com/apache/kylin/pull/174 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 (f

[GitHub] shaofengshi closed pull request #172: KYLIN-3453 Improve cube size estimation for topn, count distinct

2018-07-26 Thread GitBox
shaofengshi closed pull request #172: KYLIN-3453 Improve cube size estimation for topn,count distinct URL: https://github.com/apache/kylin/pull/172 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] asfgit commented on issue #178: KYLIN-3470 Add cache for execute and execute_output

2018-07-29 Thread GitBox
asfgit commented on issue #178: KYLIN-3470 Add cache for execute and execute_output URL: https://github.com/apache/kylin/pull/178#issuecomment-408670956 Can one of the admins verify this patch? This is an automated message fr

[GitHub] nichunen opened a new pull request #178: KYLIN-3470 Add cache for execute and execute_output

2018-07-29 Thread GitBox
nichunen opened a new pull request #178: KYLIN-3470 Add cache for execute and execute_output URL: https://github.com/apache/kylin/pull/178 To speed up list job api This is an automated message from the Apache Git Service. To

[GitHub] nichunen commented on issue #178: KYLIN-3470 Add cache for execute and execute_output

2018-07-29 Thread GitBox
nichunen commented on issue #178: KYLIN-3470 Add cache for execute and execute_output URL: https://github.com/apache/kylin/pull/178#issuecomment-408673756 retest this please This is an automated message from the Apache Git Se

[GitHub] nichunen removed a comment on issue #178: KYLIN-3470 Add cache for execute and execute_output

2018-07-29 Thread GitBox
nichunen removed a comment on issue #178: KYLIN-3470 Add cache for execute and execute_output URL: https://github.com/apache/kylin/pull/178#issuecomment-408673756 retest this please This is an automated message from the Apach

[GitHub] Emiya0306 opened a new pull request #179: KYLIN-3464 Less user confirmation

2018-07-29 Thread GitBox
Emiya0306 opened a new pull request #179: KYLIN-3464 Less user confirmation URL: https://github.com/apache/kylin/pull/179 This is an automated message from the Apache Git Service. To respond to the message, please log on GitH

[GitHub] asfgit commented on issue #179: KYLIN-3464 Less user confirmation

2018-07-29 Thread GitBox
asfgit commented on issue #179: KYLIN-3464 Less user confirmation URL: https://github.com/apache/kylin/pull/179#issuecomment-408728963 Can one of the admins verify this patch? This is an automated message from the Apache Git S

[GitHub] luguosheng1314 closed pull request #179: KYLIN-3464 Less user confirmation

2018-07-29 Thread GitBox
luguosheng1314 closed pull request #179: KYLIN-3464 Less user confirmation URL: https://github.com/apache/kylin/pull/179 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 re

[GitHub] asfgit commented on issue #180: feat(query): Make calcite connection is more configurable. #KYLIN-3475

2018-07-30 Thread GitBox
asfgit commented on issue #180: feat(query): Make calcite connection is more configurable. #KYLIN-3475 URL: https://github.com/apache/kylin/pull/180#issuecomment-408818965 Can one of the admins verify this patch? This is an a

[GitHub] ian4hu opened a new pull request #180: feat(query): Make calcite connection is more configurable. #KYLIN-3475

2018-07-30 Thread GitBox
ian4hu opened a new pull request #180: feat(query): Make calcite connection is more configurable. #KYLIN-3475 URL: https://github.com/apache/kylin/pull/180 reported by https://issues.apache.org/jira/browse/KYLIN-3475 This is

[GitHub] coveralls commented on issue #180: feat(query): Make calcite connection is more configurable. #KYLIN-3475

2018-07-30 Thread GitBox
coveralls commented on issue #180: feat(query): Make calcite connection is more configurable. #KYLIN-3475 URL: https://github.com/apache/kylin/pull/180#issuecomment-408825381 ## Pull Request Test Coverage Report for [Build 3368](https://coveralls.io/builds/18225439) * **4** of **5**

[GitHub] ian4hu commented on issue #180: feat(query): Make calcite connection is more configurable. #KYLIN-3475

2018-07-31 Thread GitBox
ian4hu commented on issue #180: feat(query): Make calcite connection is more configurable. #KYLIN-3475 URL: https://github.com/apache/kylin/pull/180#issuecomment-409157205 @shaofengshi Would you mind verifying this PR? This i

[GitHub] shaofengshi commented on issue #180: feat(query): Make calcite connection is more configurable. #KYLIN-3475

2018-07-31 Thread GitBox
shaofengshi commented on issue #180: feat(query): Make calcite connection is more configurable. #KYLIN-3475 URL: https://github.com/apache/kylin/pull/180#issuecomment-409160154 Thanks Hu! We will run the integration test for this PR; Thanks!

[GitHub] shaofengshi commented on issue #180: feat(query): Make calcite connection is more configurable. #KYLIN-3475

2018-07-31 Thread GitBox
shaofengshi commented on issue #180: feat(query): Make calcite connection is more configurable. #KYLIN-3475 URL: https://github.com/apache/kylin/pull/180#issuecomment-409192600 Compile and mvn test all good. It looks good to me. @liyang-gmt8 @rogercloud , please also take a re

[GitHub] Wayne1c opened a new pull request #181: Kylin 3471

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

[GitHub] asfgit commented on issue #181: Kylin 3471

2018-07-31 Thread GitBox
asfgit commented on issue #181: Kylin 3471 URL: https://github.com/apache/kylin/pull/181#issuecomment-409437070 Can one of the admins verify this patch? This is an automated message from the Apache Git Service. To respond to t

[GitHub] ian4hu opened a new pull request #182: KYLIN-3478 Enhance backwards compatibility.

2018-08-01 Thread GitBox
ian4hu opened a new pull request #182: KYLIN-3478 Enhance backwards compatibility. URL: https://github.com/apache/kylin/pull/182 KYLIN-3478 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] asfgit commented on issue #182: KYLIN-3478 Enhance backwards compatibility.

2018-08-01 Thread GitBox
asfgit commented on issue #182: KYLIN-3478 Enhance backwards compatibility. URL: https://github.com/apache/kylin/pull/182#issuecomment-409471076 Can one of the admins verify this patch? This is an automated message from the Ap

[GitHub] coveralls commented on issue #182: KYLIN-3478 Enhance backwards compatibility.

2018-08-01 Thread GitBox
coveralls commented on issue #182: KYLIN-3478 Enhance backwards compatibility. URL: https://github.com/apache/kylin/pull/182#issuecomment-409480700 ## Pull Request Test Coverage Report for [Build 3370](https://coveralls.io/builds/18265315) * **0** of **3** **(0.0%)** changed or ad

[GitHub] shaofengshi closed pull request #182: KYLIN-3478 Enhance backwards compatibility.

2018-08-01 Thread GitBox
shaofengshi closed pull request #182: KYLIN-3478 Enhance backwards compatibility. URL: https://github.com/apache/kylin/pull/182 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] GinaZhai opened a new pull request #183: KYLIN-3461 metastore.sh refresh-cube-signature not updating cube sign…

2018-08-01 Thread GitBox
GinaZhai opened a new pull request #183: KYLIN-3461 metastore.sh refresh-cube-signature not updating cube sign… URL: https://github.com/apache/kylin/pull/183 This is an automated message from the Apache Git Service. To respo

[GitHub] asfgit commented on issue #183: KYLIN-3461 metastore.sh refresh-cube-signature not updating cube sign…

2018-08-01 Thread GitBox
asfgit commented on issue #183: KYLIN-3461 metastore.sh refresh-cube-signature not updating cube sign… URL: https://github.com/apache/kylin/pull/183#issuecomment-409536463 Can one of the admins verify this patch? This is an a

[GitHub] shaofengshi closed pull request #183: KYLIN-3461 metastore.sh refresh-cube-signature not updating cube sign…

2018-08-01 Thread GitBox
shaofengshi closed pull request #183: KYLIN-3461 metastore.sh refresh-cube-signature not updating cube sign… URL: https://github.com/apache/kylin/pull/183 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 proven

[GitHub] shaofengshi commented on issue #180: feat(query): Make calcite connection is more configurable. #KYLIN-3475

2018-08-01 Thread GitBox
shaofengshi commented on issue #180: feat(query): Make calcite connection is more configurable. #KYLIN-3475 URL: https://github.com/apache/kylin/pull/180#issuecomment-409783317 Hi Ian, we need the commit message started with JIRA ID, for example: "KYLIN-3475 Make calcite connection configu

[GitHub] ian4hu commented on issue #180: KYLIN-3475 Make calcite connection is more configurable

2018-08-01 Thread GitBox
ian4hu commented on issue #180: KYLIN-3475 Make calcite connection is more configurable URL: https://github.com/apache/kylin/pull/180#issuecomment-409793727 @liyang-gmt8 done. This is an automated message from the Apache Git

[GitHub] shaofengshi closed pull request #180: KYLIN-3475 Make calcite connection is more configurable

2018-08-01 Thread GitBox
shaofengshi closed pull request #180: KYLIN-3475 Make calcite connection is more configurable URL: https://github.com/apache/kylin/pull/180 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

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