[GitHub] yiming187 commented on issue #102: KYLIN-2960, modify the default LDAP admin group config.

2018-02-12 Thread GitBox
yiming187 commented on issue #102: KYLIN-2960, modify the default LDAP admin group config. URL: https://github.com/apache/kylin/pull/102#issuecomment-365153978 merged This is an automated message from the Apache Git Service.

[GitHub] yiming187 closed pull request #102: KYLIN-2960, modify the default LDAP admin group config.

2018-02-12 Thread GitBox
yiming187 closed pull request #102: KYLIN-2960, modify the default LDAP admin group config. URL: https://github.com/apache/kylin/pull/102 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] yiming187 closed pull request #99: Yang21 hbase102

2018-02-12 Thread GitBox
yiming187 closed pull request #99: Yang21 hbase102 URL: https://github.com/apache/kylin/pull/99 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above t

[GitHub] yiming187 closed pull request #67: Fix the double carets bug in sample.sh for old bash version

2018-02-12 Thread GitBox
yiming187 closed pull request #67: Fix the double carets bug in sample.sh for old bash version URL: https://github.com/apache/kylin/pull/67 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] yiming187 commented on issue #67: Fix the double carets bug in sample.sh for old bash version

2018-02-12 Thread GitBox
yiming187 commented on issue #67: Fix the double carets bug in sample.sh for old bash version URL: https://github.com/apache/kylin/pull/67#issuecomment-365154160 merged. This is an automated message from the Apache Git Servi

[GitHub] yiming187 closed pull request #58: kylin sso????cas?????

2018-02-12 Thread GitBox
yiming187 closed pull request #58: kylin ssocas? URL: https://github.com/apache/kylin/pull/58 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL a

[GitHub] yiming187 closed pull request #33: ????

2018-02-12 Thread GitBox
yiming187 closed pull request #33: URL: https://github.com/apache/kylin/pull/33 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the

[GitHub] yiming187 closed pull request #40: 1.3.x

2018-02-12 Thread GitBox
yiming187 closed pull request #40: 1.3.x URL: https://github.com/apache/kylin/pull/40 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to th

[GitHub] yiming187 closed pull request #42: Update pom.xml

2018-02-12 Thread GitBox
yiming187 closed pull request #42: Update pom.xml URL: https://github.com/apache/kylin/pull/42 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to

[GitHub] yiming187 closed pull request #60: support jdbc source

2018-02-12 Thread GitBox
yiming187 closed pull request #60: support jdbc source URL: https://github.com/apache/kylin/pull/60 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL abo

[GitHub] yiming187 closed pull request #61: Fix for KYLIN-2668

2018-02-12 Thread GitBox
yiming187 closed pull request #61: Fix for KYLIN-2668 URL: https://github.com/apache/kylin/pull/61 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), t

[GitHub] yiming187 closed pull request #64: JIRA_ID:KYLIN-2819

2018-02-12 Thread GitBox
yiming187 closed pull request #64: JIRA_ID:KYLIN-2819 URL: https://github.com/apache/kylin/pull/64 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), t

[GitHub] yiming187 closed pull request #94: 2.2.x

2018-02-12 Thread GitBox
yiming187 closed pull request #94: 2.2.x URL: https://github.com/apache/kylin/pull/94 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to th

[GitHub] yiming187 closed pull request #44: change the style of return that make it read easy

2018-02-12 Thread GitBox
yiming187 closed pull request #44: change the style of return that make it read easy URL: https://github.com/apache/kylin/pull/44 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 forei

[GitHub] yiming187 opened a new pull request #103: enable coveralls

2018-02-13 Thread GitBox
yiming187 opened a new pull request #103: enable coveralls URL: https://github.com/apache/kylin/pull/103 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the UR

[GitHub] asfgit commented on issue #103: enable coveralls

2018-02-13 Thread GitBox
asfgit commented on issue #103: enable coveralls URL: https://github.com/apache/kylin/pull/103#issuecomment-365270991 Can one of the admins verify this patch? This is an automated message from the Apache Git Service. To respon

[GitHub] yiming187 closed pull request #103: enable coveralls

2018-02-15 Thread GitBox
yiming187 closed pull request #103: enable coveralls URL: https://github.com/apache/kylin/pull/103 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), t

[GitHub] asfgit commented on issue #103: enable coveralls

2018-02-15 Thread GitBox
asfgit commented on issue #103: enable coveralls URL: https://github.com/apache/kylin/pull/103#issuecomment-365884277 Can one of the admins verify this patch? This is an automated message from the Apache Git Service. To respon

[GitHub] yiming187 opened a new pull request #103: enable coveralls

2018-02-15 Thread GitBox
yiming187 opened a new pull request #103: enable coveralls URL: https://github.com/apache/kylin/pull/103 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the UR

[GitHub] yiming187 closed pull request #103: enable coveralls

2018-02-15 Thread GitBox
yiming187 closed pull request #103: enable coveralls URL: https://github.com/apache/kylin/pull/103 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), t

[GitHub] codecov-io commented on issue #104: KYLIN-3263, bugfix with AbstractExecutable's retry.

2018-02-23 Thread GitBox
codecov-io commented on issue #104: KYLIN-3263, bugfix with AbstractExecutable's retry. URL: https://github.com/apache/kylin/pull/104#issuecomment-367982611 # [Codecov](https://codecov.io/gh/apache/kylin/pull/104?src=pr&el=h1) Report > Merging [#104](https://codecov.io/gh/apache/kylin/pu

[GitHub] coveralls commented on issue #104: KYLIN-3263, bugfix with AbstractExecutable's retry.

2018-02-23 Thread GitBox
coveralls commented on issue #104: KYLIN-3263, bugfix with AbstractExecutable's retry. URL: https://github.com/apache/kylin/pull/104#issuecomment-367983164 ## Pull Request Test Coverage Report for [Build 2991](https://coveralls.io/builds/15663717) * **5** of **5** **(100.0%)** chan

[GitHub] yiming187 closed pull request #104: KYLIN-3263, bugfix with AbstractExecutable's retry.

2018-02-23 Thread GitBox
yiming187 closed pull request #104: KYLIN-3263, bugfix with AbstractExecutable's retry. URL: https://github.com/apache/kylin/pull/104 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 f

[GitHub] asfgit commented on issue #105: KYLIN-2884 Add delete segment function for portal - bug fix

2018-02-23 Thread GitBox
asfgit commented on issue #105: KYLIN-2884 Add delete segment function for portal - bug fix URL: https://github.com/apache/kylin/pull/105#issuecomment-36820 Can one of the admins verify this patch? This is an automated me

[GitHub] sanjulian opened a new pull request #105: KYLIN-2884 Add delete segment function for portal - bug fix

2018-02-23 Thread GitBox
sanjulian opened a new pull request #105: KYLIN-2884 Add delete segment function for portal - bug fix URL: https://github.com/apache/kylin/pull/105 This is an automated message from the Apache Git Service. To respond to the

[GitHub] coveralls commented on issue #105: KYLIN-2884 Add delete segment function for portal - bug fix

2018-02-23 Thread GitBox
coveralls commented on issue #105: KYLIN-2884 Add delete segment function for portal - bug fix URL: https://github.com/apache/kylin/pull/105#issuecomment-368205238 ## Pull Request Test Coverage Report for [Build 2997](https://coveralls.io/builds/15677654) * **0** of **0** **(NaN%)*

[GitHub] codecov-io commented on issue #105: KYLIN-2884 Add delete segment function for portal - bug fix

2018-02-23 Thread GitBox
codecov-io commented on issue #105: KYLIN-2884 Add delete segment function for portal - bug fix URL: https://github.com/apache/kylin/pull/105#issuecomment-368205395 # [Codecov](https://codecov.io/gh/apache/kylin/pull/105?src=pr&el=h1) Report > Merging [#105](https://codecov.io/gh/apache/

[GitHub] chenzhx closed pull request #105: KYLIN-2884 Add delete segment function for portal - bug fix

2018-02-24 Thread GitBox
chenzhx closed pull request #105: KYLIN-2884 Add delete segment function for portal - bug fix URL: https://github.com/apache/kylin/pull/105 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

[GitHub] asfgit commented on issue #106: When I connect kylin with JDBC, the query date has a problem.

2018-02-24 Thread GitBox
asfgit commented on issue #106: When I connect kylin with JDBC, the query date has a problem. URL: https://github.com/apache/kylin/pull/106#issuecomment-368213461 Can one of the admins verify this patch? This is an automated

[GitHub] starbow727 opened a new pull request #106: When I connect kylin with JDBC, the query date has a problem.

2018-02-24 Thread GitBox
starbow727 opened a new pull request #106: When I connect kylin with JDBC, the query date has a problem. URL: https://github.com/apache/kylin/pull/106 hello, When I connect kylin with JDBC, the query date has a problem. The query condition is WHERE td.ca

[GitHub] yiming187 commented on issue #106: When I connect kylin with JDBC, the query date has a problem.

2018-02-24 Thread GitBox
yiming187 commented on issue #106: When I connect kylin with JDBC, the query date has a problem. URL: https://github.com/apache/kylin/pull/106#issuecomment-368216841 Hi @starbow727, for questions, please ask it in kylin user mailer, not github PR. Cloud you close this PR and subscribe to k

[GitHub] pengjianhua opened a new pull request #107: kylin-3119

2018-02-26 Thread GitBox
pengjianhua opened a new pull request #107: kylin-3119 URL: https://github.com/apache/kylin/pull/107 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL ab

[GitHub] asfgit commented on issue #107: kylin-3119

2018-02-26 Thread GitBox
asfgit commented on issue #107: kylin-3119 URL: https://github.com/apache/kylin/pull/107#issuecomment-368736036 Can one of the admins verify this patch? This is an automated message from the Apache Git Service. To respond to t

[GitHub] coveralls commented on issue #107: kylin-3119

2018-02-26 Thread GitBox
coveralls commented on issue #107: kylin-3119 URL: https://github.com/apache/kylin/pull/107#issuecomment-368738219 ## Pull Request Test Coverage Report for [Build 2999](https://coveralls.io/builds/15711288) * **16** of **16** **(100.0%)** changed or added relevant lines in **1** fi

[GitHub] codecov-io commented on issue #107: kylin-3119

2018-02-26 Thread GitBox
codecov-io commented on issue #107: kylin-3119 URL: https://github.com/apache/kylin/pull/107#issuecomment-368738780 # [Codecov](https://codecov.io/gh/apache/kylin/pull/107?src=pr&el=h1) Report > Merging [#107](https://codecov.io/gh/apache/kylin/pull/107?src=pr&el=desc) into [master](htt

[GitHub] pengjianhua commented on issue #107: kylin-3119

2018-02-26 Thread GitBox
pengjianhua commented on issue #107: kylin-3119 URL: https://github.com/apache/kylin/pull/107#issuecomment-368739294 https://issues.apache.org/jira/browse/KYLIN-3119 This is an automated message from the Apache Git Service. To

[GitHub] rogercloud commented on a change in pull request #107: kylin-3119

2018-02-26 Thread GitBox
rogercloud commented on a change in pull request #107: kylin-3119 URL: https://github.com/apache/kylin/pull/107#discussion_r170827219 ## File path: query/src/main/java/org/apache/kylin/query/util/QueryUtil.java ## @@ -50,32 +50,39 @@ public static String massageSql(String s

[GitHub] rogercloud commented on a change in pull request #107: kylin-3119

2018-02-26 Thread GitBox
rogercloud commented on a change in pull request #107: kylin-3119 URL: https://github.com/apache/kylin/pull/107#discussion_r170828728 ## File path: query/src/test/java/org/apache/kylin/query/util/QueryUtilTest.java ## @@ -51,6 +51,58 @@ public void testMassageSql() {

[GitHub] asfgit commented on issue #108: Apache kylin 2895

2018-02-26 Thread GitBox
asfgit commented on issue #108: Apache kylin 2895 URL: https://github.com/apache/kylin/pull/108#issuecomment-368778494 Can one of the admins verify this patch? This is an automated message from the Apache Git Service. To respo

[GitHub] kyotoYaho opened a new pull request #108: Apache kylin 2895

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

[GitHub] pengjianhua commented on issue #107: kylin-3119

2018-02-27 Thread GitBox
pengjianhua commented on issue #107: kylin-3119 URL: https://github.com/apache/kylin/pull/107#issuecomment-369110791 hi,rogercloud.the `kylinConfig.getForceLimit()` add a check too. And the reason that I use "line.separator" is in function massageSql has the code `sql = sql.replace

[GitHub] coveralls commented on issue #107: kylin-3119

2018-02-27 Thread GitBox
coveralls commented on issue #107: kylin-3119 URL: https://github.com/apache/kylin/pull/107#issuecomment-368738219 ## Pull Request Test Coverage Report for [Build 3001](https://coveralls.io/builds/15731959) * **16** of **16** **(100.0%)** changed or added relevant lines in **1** fi

[GitHub] codecov-io commented on issue #107: kylin-3119

2018-02-27 Thread GitBox
codecov-io commented on issue #107: kylin-3119 URL: https://github.com/apache/kylin/pull/107#issuecomment-368738780 # [Codecov](https://codecov.io/gh/apache/kylin/pull/107?src=pr&el=h1) Report > Merging [#107](https://codecov.io/gh/apache/kylin/pull/107?src=pr&el=desc) into [master](htt

[GitHub] asfgit commented on issue #109: KYLIN-3265 Add jobSearchMode as a filter condition for jobs UI part

2018-02-28 Thread GitBox
asfgit commented on issue #109: KYLIN-3265 Add jobSearchMode as a filter condition for jobs UI part URL: https://github.com/apache/kylin/pull/109#issuecomment-369451560 Can one of the admins verify this patch? This is an auto

[GitHub] chenzhx opened a new pull request #109: KYLIN-3265 Add jobSearchMode as a filter condition for jobs UI part

2018-02-28 Thread GitBox
chenzhx opened a new pull request #109: KYLIN-3265 Add jobSearchMode as a filter condition for jobs UI part URL: https://github.com/apache/kylin/pull/109 Signed-off-by: chenzhx This is an automated message from the Apache Gi

[GitHub] chenzhx closed pull request #109: KYLIN-3265 Add jobSearchMode as a filter condition for jobs UI part

2018-02-28 Thread GitBox
chenzhx closed pull request #109: KYLIN-3265 Add jobSearchMode as a filter condition for jobs UI part URL: https://github.com/apache/kylin/pull/109 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] codecov-io commented on issue #107: kylin-3119

2018-02-28 Thread GitBox
codecov-io commented on issue #107: kylin-3119 URL: https://github.com/apache/kylin/pull/107#issuecomment-368738780 # [Codecov](https://codecov.io/gh/apache/kylin/pull/107?src=pr&el=h1) Report > Merging [#107](https://codecov.io/gh/apache/kylin/pull/107?src=pr&el=desc) into [master](htt

[GitHub] coveralls commented on issue #107: kylin-3119

2018-02-28 Thread GitBox
coveralls commented on issue #107: kylin-3119 URL: https://github.com/apache/kylin/pull/107#issuecomment-368738219 ## Pull Request Test Coverage Report for [Build 3008](https://coveralls.io/builds/15751320) * **16** of **16** **(100.0%)** changed or added relevant lines in **1** fi

[GitHub] codecov-io commented on issue #109: KYLIN-3265 Add jobSearchMode as a filter condition for jobs UI part

2018-02-28 Thread GitBox
codecov-io commented on issue #109: KYLIN-3265 Add jobSearchMode as a filter condition for jobs UI part URL: https://github.com/apache/kylin/pull/109#issuecomment-369454646 # [Codecov](https://codecov.io/gh/apache/kylin/pull/109?src=pr&el=h1) Report > Merging [#109](https://codecov.io/gh

[GitHub] coveralls commented on issue #109: KYLIN-3265 Add jobSearchMode as a filter condition for jobs UI part

2018-02-28 Thread GitBox
coveralls commented on issue #109: KYLIN-3265 Add jobSearchMode as a filter condition for jobs UI part URL: https://github.com/apache/kylin/pull/109#issuecomment-369454934 ## Pull Request Test Coverage Report for [Build 3009](https://coveralls.io/builds/15751349) * **0** of **0** *

[GitHub] asfgit commented on issue #110: KYLIN-3236

2018-03-02 Thread GitBox
asfgit commented on issue #110: KYLIN-3236 URL: https://github.com/apache/kylin/pull/110#issuecomment-369859958 Can one of the admins verify this patch? This is an automated message from the Apache Git Service. To respond to t

[GitHub] xingpeng1 opened a new pull request #110: KYLIN-3236

2018-03-02 Thread GitBox
xingpeng1 opened a new pull request #110: KYLIN-3236 URL: https://github.com/apache/kylin/pull/110 KYLIN-3236 The function 'reGenerateAdvancedDict()' has an error logical judgment, which will cause an exception when you edit the cube. Signed-off-by: xingpeng1 -

[GitHub] yiming187 commented on issue #106: When I connect kylin with JDBC, the query date has a problem.

2018-03-05 Thread GitBox
yiming187 commented on issue #106: When I connect kylin with JDBC, the query date has a problem. URL: https://github.com/apache/kylin/pull/106#issuecomment-370492412 Close it, as it's an invalid PR. This is an automated mess

[GitHub] yiming187 closed pull request #106: When I connect kylin with JDBC, the query date has a problem.

2018-03-05 Thread GitBox
yiming187 closed pull request #106: When I connect kylin with JDBC, the query date has a problem. URL: https://github.com/apache/kylin/pull/106 This is an automated message from the Apache Git Service. To respond to the mess

[GitHub] tttMelody opened a new pull request #111: ACL enhancement.

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

[GitHub] asfgit commented on issue #111: ACL enhancement.

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

[GitHub] codecov-io commented on issue #111: ACL enhancement.

2018-03-05 Thread GitBox
codecov-io commented on issue #111: ACL enhancement. URL: https://github.com/apache/kylin/pull/111#issuecomment-370668481 # [Codecov](https://codecov.io/gh/apache/kylin/pull/111?src=pr&el=h1) Report > Merging [#111](https://codecov.io/gh/apache/kylin/pull/111?src=pr&el=desc) into [maste

[GitHub] coveralls commented on issue #111: ACL enhancement.

2018-03-05 Thread GitBox
coveralls commented on issue #111: ACL enhancement. URL: https://github.com/apache/kylin/pull/111#issuecomment-370668572 ## Pull Request Test Coverage Report for [Build 3026](https://coveralls.io/builds/15823854) * **3** of **515** **(0.58%)** changed or added relevant lines in **19

[GitHub] codecov-io commented on issue #111: ACL enhancement.

2018-03-06 Thread GitBox
codecov-io commented on issue #111: ACL enhancement. URL: https://github.com/apache/kylin/pull/111#issuecomment-370668481 # [Codecov](https://codecov.io/gh/apache/kylin/pull/111?src=pr&el=h1) Report > Merging [#111](https://codecov.io/gh/apache/kylin/pull/111?src=pr&el=desc) into [maste

[GitHub] coveralls commented on issue #111: ACL enhancement.

2018-03-06 Thread GitBox
coveralls commented on issue #111: ACL enhancement. URL: https://github.com/apache/kylin/pull/111#issuecomment-370668572 ## Pull Request Test Coverage Report for [Build 3027](https://coveralls.io/builds/15828627) * **3** of **515** **(0.58%)** changed or added relevant lines in **19

[GitHub] tttMelody commented on issue #111: ACL enhancement.

2018-03-06 Thread GitBox
tttMelody commented on issue #111: ACL enhancement. URL: https://github.com/apache/kylin/pull/111#issuecomment-370774979 retest this please. This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] yiming187 closed pull request #111: ACL enhancement.

2018-03-06 Thread GitBox
yiming187 closed pull request #111: ACL enhancement. URL: https://github.com/apache/kylin/pull/111 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), t

[GitHub] asfgit commented on issue #112: Merge pull request #1 from apache/master

2018-03-07 Thread GitBox
asfgit commented on issue #112: Merge pull request #1 from apache/master URL: https://github.com/apache/kylin/pull/112#issuecomment-371382910 Can one of the admins verify this patch? This is an automated message from the Apach

[GitHub] eissac opened a new pull request #112: Merge pull request #1 from apache/master

2018-03-07 Thread GitBox
eissac opened a new pull request #112: Merge pull request #1 from apache/master URL: https://github.com/apache/kylin/pull/112 syncing a fork This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] eissac closed pull request #112: Merge pull request #1 from apache/master

2018-03-07 Thread GitBox
eissac closed pull request #112: Merge pull request #1 from apache/master URL: https://github.com/apache/kylin/pull/112 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 req

[GitHub] coveralls commented on issue #112: Merge pull request #1 from apache/master

2018-03-07 Thread GitBox
coveralls commented on issue #112: Merge pull request #1 from apache/master URL: https://github.com/apache/kylin/pull/112#issuecomment-371385216 ## Pull Request Test Coverage Report for [Build 3029](https://coveralls.io/builds/15868118) * **0** of **0** **(NaN%)** changed or added r

[GitHub] codecov-io commented on issue #112: Merge pull request #1 from apache/master

2018-03-07 Thread GitBox
codecov-io commented on issue #112: Merge pull request #1 from apache/master URL: https://github.com/apache/kylin/pull/112#issuecomment-371385736 # [Codecov](https://codecov.io/gh/apache/kylin/pull/112?src=pr&el=h1) Report > Merging [#112](https://codecov.io/gh/apache/kylin/pull/112?src=p

[GitHub] asfgit commented on issue #113: KYLIN-3280

2018-03-08 Thread GitBox
asfgit commented on issue #113: KYLIN-3280 URL: https://github.com/apache/kylin/pull/113#issuecomment-371694247 Can one of the admins verify this patch? This is an automated message from the Apache Git Service. To respond to t

[GitHub] xingpeng1 opened a new pull request #113: KYLIN-3280

2018-03-08 Thread GitBox
xingpeng1 opened a new pull request #113: KYLIN-3280 URL: https://github.com/apache/kylin/pull/113 KYLIN-3280 The delete button should not be enabled without any segment in cube segment delete confirm dialog This is an automa

[GitHub] coveralls commented on issue #113: KYLIN-3280

2018-03-08 Thread GitBox
coveralls commented on issue #113: KYLIN-3280 URL: https://github.com/apache/kylin/pull/113#issuecomment-371696607 ## Pull Request Test Coverage Report for [Build 3030](https://coveralls.io/builds/15887406) * **0** of **0** **(NaN%)** changed or added relevant lines in **0** files

[GitHub] codecov-io commented on issue #113: KYLIN-3280

2018-03-08 Thread GitBox
codecov-io commented on issue #113: KYLIN-3280 URL: https://github.com/apache/kylin/pull/113#issuecomment-371697041 # [Codecov](https://codecov.io/gh/apache/kylin/pull/113?src=pr&el=h1) Report > Merging [#113](https://codecov.io/gh/apache/kylin/pull/113?src=pr&el=desc) into [master](htt

[GitHub] shaofengshi commented on issue #113: KYLIN-3280

2018-03-11 Thread GitBox
shaofengshi commented on issue #113: KYLIN-3280 URL: https://github.com/apache/kylin/pull/113#issuecomment-372112662 As there is just 1 line code change, can you merge the 3 commits to 1? thanks This is an automated message fr

[GitHub] coveralls commented on issue #113: KYLIN-3280

2018-03-11 Thread GitBox
coveralls commented on issue #113: KYLIN-3280 URL: https://github.com/apache/kylin/pull/113#issuecomment-371696607 ## Pull Request Test Coverage Report for [Build 3034](https://coveralls.io/builds/15920496) * **0** of **0** **(NaN%)** changed or added relevant lines in **0** files

[GitHub] codecov-io commented on issue #113: KYLIN-3280

2018-03-11 Thread GitBox
codecov-io commented on issue #113: KYLIN-3280 URL: https://github.com/apache/kylin/pull/113#issuecomment-371697041 # [Codecov](https://codecov.io/gh/apache/kylin/pull/113?src=pr&el=h1) Report > Merging [#113](https://codecov.io/gh/apache/kylin/pull/113?src=pr&el=desc) into [master](htt

[GitHub] xingpeng1 commented on issue #113: KYLIN-3280

2018-03-11 Thread GitBox
xingpeng1 commented on issue #113: KYLIN-3280 URL: https://github.com/apache/kylin/pull/113#issuecomment-372183815 @shaofengshi , I have modified, there is only one commit for this PR, thanks for your reminder. This is an aut

[GitHub] xingpeng1 opened a new pull request #113: KYLIN-3280

2018-03-11 Thread GitBox
xingpeng1 opened a new pull request #113: KYLIN-3280 URL: https://github.com/apache/kylin/pull/113 KYLIN-3280 The delete button should not be enabled without any segment in cube segment delete confirm dialog This is an automa

[GitHub] asfgit commented on issue #113: KYLIN-3280

2018-03-11 Thread GitBox
asfgit commented on issue #113: KYLIN-3280 URL: https://github.com/apache/kylin/pull/113#issuecomment-372183900 Can one of the admins verify this patch? This is an automated message from the Apache Git Service. To respond to t

[GitHub] xingpeng1 closed pull request #113: KYLIN-3280

2018-03-11 Thread GitBox
xingpeng1 closed pull request #113: KYLIN-3280 URL: https://github.com/apache/kylin/pull/113 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 dif

[GitHub] coveralls commented on issue #113: KYLIN-3280

2018-03-11 Thread GitBox
coveralls commented on issue #113: KYLIN-3280 URL: https://github.com/apache/kylin/pull/113#issuecomment-371696607 ## Pull Request Test Coverage Report for [Build 3035](https://coveralls.io/builds/15920601) * **0** of **0** **(NaN%)** changed or added relevant lines in **0** files

[GitHub] codecov-io commented on issue #113: KYLIN-3280

2018-03-11 Thread GitBox
codecov-io commented on issue #113: KYLIN-3280 URL: https://github.com/apache/kylin/pull/113#issuecomment-371697041 # [Codecov](https://codecov.io/gh/apache/kylin/pull/113?src=pr&el=h1) Report > Merging [#113](https://codecov.io/gh/apache/kylin/pull/113?src=pr&el=desc) into [master](htt

[GitHub] tttMelody opened a new pull request #114: KYLIN-3287, fix when a shard by column is in dict encoding, dict buil?

2018-03-12 Thread GitBox
tttMelody opened a new pull request #114: KYLIN-3287, fix when a shard by column is in dict encoding, dict buil? URL: https://github.com/apache/kylin/pull/114 ?ding error. This is an automated message from the Apache Git Serv

[GitHub] asfgit commented on issue #114: KYLIN-3287, fix when a shard by column is in dict encoding, dict buil?

2018-03-12 Thread GitBox
asfgit commented on issue #114: KYLIN-3287, fix when a shard by column is in dict encoding, dict buil? URL: https://github.com/apache/kylin/pull/114#issuecomment-372291210 Can one of the admins verify this patch? This is an a

[GitHub] liyang-gmt8 closed pull request #114: KYLIN-3287, fix when a shard by column is in dict encoding, dict buil?

2018-03-12 Thread GitBox
liyang-gmt8 closed pull request #114: KYLIN-3287, fix when a shard by column is in dict encoding, dict buil? URL: https://github.com/apache/kylin/pull/114 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] codecov-io commented on issue #114: KYLIN-3287, fix when a shard by column is in dict encoding, dict buil?

2018-03-12 Thread GitBox
codecov-io commented on issue #114: KYLIN-3287, fix when a shard by column is in dict encoding, dict buil? URL: https://github.com/apache/kylin/pull/114#issuecomment-372295252 # [Codecov](https://codecov.io/gh/apache/kylin/pull/114?src=pr&el=h1) Report > Merging [#114](https://codecov.io

[GitHub] coveralls commented on issue #114: KYLIN-3287, fix when a shard by column is in dict encoding, dict buil?

2018-03-12 Thread GitBox
coveralls commented on issue #114: KYLIN-3287, fix when a shard by column is in dict encoding, dict buil? URL: https://github.com/apache/kylin/pull/114#issuecomment-372295747 ## Pull Request Test Coverage Report for [Build 3036](https://coveralls.io/builds/15926510) * **0** of **1**

[GitHub] yiming187 closed pull request #107: kylin-3119

2018-03-12 Thread GitBox
yiming187 closed pull request #107: kylin-3119 URL: https://github.com/apache/kylin/pull/107 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 dif

[GitHub] yiming187 closed pull request #113: KYLIN-3280

2018-03-12 Thread GitBox
yiming187 closed pull request #113: KYLIN-3280 URL: https://github.com/apache/kylin/pull/113 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 dif

[GitHub] asfgit commented on issue #115: minor, refine exception handling.

2018-03-13 Thread GitBox
asfgit commented on issue #115: minor, refine exception handling. URL: https://github.com/apache/kylin/pull/115#issuecomment-372889290 Can one of the admins verify this patch? This is an automated message from the Apache Git S

[GitHub] tttMelody opened a new pull request #115: minor, refine exception handling.

2018-03-13 Thread GitBox
tttMelody opened a new pull request #115: minor, refine exception handling. URL: https://github.com/apache/kylin/pull/115 This is an automated message from the Apache Git Service. To respond to the message, please log on GitH

[GitHub] coveralls commented on issue #115: minor, refine exception handling.

2018-03-13 Thread GitBox
coveralls commented on issue #115: minor, refine exception handling. URL: https://github.com/apache/kylin/pull/115#issuecomment-372891622 ## Pull Request Test Coverage Report for [Build 3045](https://coveralls.io/builds/15963074) * **0** of **1** **(0.0%)** changed or added relevant

[GitHub] yiming187 closed pull request #115: minor, refine exception handling.

2018-03-14 Thread GitBox
yiming187 closed pull request #115: minor, refine exception handling. URL: https://github.com/apache/kylin/pull/115 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

[GitHub] xingpeng1 opened a new pull request #116: KYLIN-3292

2018-03-16 Thread GitBox
xingpeng1 opened a new pull request #116: KYLIN-3292 URL: https://github.com/apache/kylin/pull/116 KYLIN-3292 The update button should be disabled with none key and value in setting config dialog This is an automated message

[GitHub] asfgit commented on issue #116: KYLIN-3292

2018-03-16 Thread GitBox
asfgit commented on issue #116: KYLIN-3292 URL: https://github.com/apache/kylin/pull/116#issuecomment-373620961 Can one of the admins verify this patch? This is an automated message from the Apache Git Service. To respond to t

[GitHub] coveralls commented on issue #116: KYLIN-3292

2018-03-16 Thread GitBox
coveralls commented on issue #116: KYLIN-3292 URL: https://github.com/apache/kylin/pull/116#issuecomment-373624959 ## Pull Request Test Coverage Report for [Build 3048](https://coveralls.io/builds/16010769) * **0** of **0** **(NaN%)** changed or added relevant lines in **0** files

[GitHub] asfgit commented on issue #117: KYLIN-3293, fix FixedLenHexDimEnc.java return a wrong code length lea…

2018-03-16 Thread GitBox
asfgit commented on issue #117: KYLIN-3293, fix FixedLenHexDimEnc.java return a wrong code length lea… URL: https://github.com/apache/kylin/pull/117#issuecomment-373730001 Can one of the admins verify this patch? This is an a

[GitHub] tttMelody opened a new pull request #117: KYLIN-3293, fix FixedLenHexDimEnc.java return a wrong code length lea…

2018-03-16 Thread GitBox
tttMelody opened a new pull request #117: KYLIN-3293, fix FixedLenHexDimEnc.java return a wrong code length lea… URL: https://github.com/apache/kylin/pull/117 …ds to cut bytes error. This is an automated message from the Apac

[GitHub] codecov-io commented on issue #117: KYLIN-3293, fix FixedLenHexDimEnc.java return a wrong code length lea…

2018-03-16 Thread GitBox
codecov-io commented on issue #117: KYLIN-3293, fix FixedLenHexDimEnc.java return a wrong code length lea… URL: https://github.com/apache/kylin/pull/117#issuecomment-373735030 # [Codecov](https://codecov.io/gh/apache/kylin/pull/117?src=pr&el=h1) Report > Merging [#117](https://codecov.io

[GitHub] coveralls commented on issue #117: KYLIN-3293, fix FixedLenHexDimEnc.java return a wrong code length lea…

2018-03-16 Thread GitBox
coveralls commented on issue #117: KYLIN-3293, fix FixedLenHexDimEnc.java return a wrong code length lea… URL: https://github.com/apache/kylin/pull/117#issuecomment-373735894 ## Pull Request Test Coverage Report for [Build 3049](https://coveralls.io/builds/16016963) * **6** of **9**

[GitHub] asfgit commented on issue #118: Kyligence weekly update

2018-03-18 Thread GitBox
asfgit commented on issue #118: Kyligence weekly update URL: https://github.com/apache/kylin/pull/118#issuecomment-373979533 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 #118: Kyligence weekly update

2018-03-18 Thread GitBox
liyang-gmt8 opened a new pull request #118: Kyligence weekly update URL: https://github.com/apache/kylin/pull/118 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and u

  1   2   3   4   5   6   7   8   9   10   >