[GitHub] [kylin] coveralls commented on issue #621: KYLIN-3987 Set a larger value of reducer num for fact distinct job

coveralls commented on issue #621: KYLIN-3987 Set a larger value of reducer num for fact distinct job URL: https://github.com/apache/kylin/pull/621#issuecomment-487019097 ## Pull Request Test Coverage Report for [Build 4425](https://coveralls.io/builds/23042291) * **0** of **1** *

[GitHub] [kylin] nichunen merged pull request #621: KYLIN-3987 Set a larger value of reducer num for fact distinct job

nichunen merged pull request #621: KYLIN-3987 Set a larger value of reducer num for fact distinct job URL: https://github.com/apache/kylin/pull/621 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [kylin] nichunen merged pull request #620: KYLIN-3981 Auto Merge Job failed to execute on windows

nichunen merged pull request #620: KYLIN-3981 Auto Merge Job failed to execute on windows URL: https://github.com/apache/kylin/pull/620 This is an automated message from the Apache Git Service. To respond to the message, ple

[GitHub] [kylin] GinaZhai opened a new pull request #622: Kylin 3893 Add validation for the encoding length and the type of encoding

GinaZhai opened a new pull request #622: Kylin 3893 Add validation for the encoding length and the type of encoding URL: https://github.com/apache/kylin/pull/622 This is an automated message from the Apache Git Service. To r

[GitHub] [kylin] asfgit commented on issue #622: Kylin 3893 Add validation for the encoding length and the type of encoding

asfgit commented on issue #622: Kylin 3893 Add validation for the encoding length and the type of encoding URL: https://github.com/apache/kylin/pull/622#issuecomment-487297766 Can one of the admins verify this patch? This is

[GitHub] [kylin] codecov-io commented on issue #622: KYLIN-3893 Add validation for the encoding length and the type of encoding

codecov-io commented on issue #622: KYLIN-3893 Add validation for the encoding length and the type of encoding URL: https://github.com/apache/kylin/pull/622#issuecomment-487299929 # [Codecov](https://codecov.io/gh/apache/kylin/pull/622?src=pr&el=h1) Report > Merging [#622](https://codeco

[GitHub] [kylin] coveralls commented on issue #622: KYLIN-3893 Add validation for the encoding length and the type of encoding

coveralls commented on issue #622: KYLIN-3893 Add validation for the encoding length and the type of encoding URL: https://github.com/apache/kylin/pull/622#issuecomment-487300031 ## Pull Request Test Coverage Report for [Build 4427](https://coveralls.io/builds/23062942) * **2** of *

[GitHub] [kylin] nichunen commented on issue #534: KYLIN-3893: Cube build failed for wrong row key column description

nichunen commented on issue #534: KYLIN-3893: Cube build failed for wrong row key column description URL: https://github.com/apache/kylin/pull/534#issuecomment-487363568 Will be done in https://github.com/apache/kylin/pull/622, close this PR

[GitHub] [kylin] majic31 closed pull request #614: Fixed use customized precision

majic31 closed pull request #614: Fixed use customized precision URL: https://github.com/apache/kylin/pull/614 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and u

[GitHub] [kylin] nichunen closed pull request #534: KYLIN-3893: Cube build failed for wrong row key column description

nichunen closed pull request #534: KYLIN-3893: Cube build failed for wrong row key column description URL: https://github.com/apache/kylin/pull/534 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [kylin] majic31 opened a new pull request #623: KYLIN-3968 Customized precision doesn't work in web

majic31 opened a new pull request #623: KYLIN-3968 Customized precision doesn't work in web URL: https://github.com/apache/kylin/pull/623 In the cubeMeasures.js, It will withdraw precision and scale by using Regular Expression. The scale parameter is ok, but precision use the magic numbe

[GitHub] [kylin] asfgit commented on issue #623: KYLIN-3968 Customized precision doesn't work in web

asfgit commented on issue #623: KYLIN-3968 Customized precision doesn't work in web URL: https://github.com/apache/kylin/pull/623#issuecomment-487368541 Can one of the admins verify this patch? This is an automated message fr

[GitHub] [kylin] codecov-io commented on issue #623: KYLIN-3968 Customized precision doesn't work in web

codecov-io commented on issue #623: KYLIN-3968 Customized precision doesn't work in web URL: https://github.com/apache/kylin/pull/623#issuecomment-487370578 # [Codecov](https://codecov.io/gh/apache/kylin/pull/623?src=pr&el=h1) Report > Merging [#623](https://codecov.io/gh/apache/kylin/pu

[GitHub] [kylin] coveralls commented on issue #623: KYLIN-3968 Customized precision doesn't work in web

coveralls commented on issue #623: KYLIN-3968 Customized precision doesn't work in web URL: https://github.com/apache/kylin/pull/623#issuecomment-487371007 ## Pull Request Test Coverage Report for [Build 4429](https://coveralls.io/builds/23069287) * **0** of **0** changed or added

[GitHub] [kylin] GuoNingNing opened a new pull request #624: Allow user-specified time format using real-time

GuoNingNing opened a new pull request #624: Allow user-specified time format using real-time URL: https://github.com/apache/kylin/pull/624 Allow user-specified time format using real-time If your data source is : `{ "timestamp":"2019-04-29 11:11:11","gmv":1.1 }` You can specify

[GitHub] [kylin] asfgit commented on issue #624: Allow user-specified time format using real-time

asfgit commented on issue #624: Allow user-specified time format using real-time URL: https://github.com/apache/kylin/pull/624#issuecomment-487491048 Can one of the admins verify this patch? This is an automated message from t

[GitHub] [kylin] nichunen merged pull request #623: KYLIN-3968 Customized precision doesn't work in web

nichunen merged pull request #623: KYLIN-3968 Customized precision doesn't work in web URL: https://github.com/apache/kylin/pull/623 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [kylin] nichunen commented on issue #582: Read password from file for Beeline

nichunen commented on issue #582: Read password from file for Beeline URL: https://github.com/apache/kylin/pull/582#issuecomment-487801156 Jenkins, test this please. This is an automated message from the Apache Git Service. To

[GitHub] [kylin] nichunen commented on issue #582: Read password from file for Beeline

nichunen commented on issue #582: Read password from file for Beeline URL: https://github.com/apache/kylin/pull/582#issuecomment-487806832 @dekarlab Hi, your pr build failed due to [ERROR] Forbidden class/interface use: java.io.FileReader [Uses default charset] [ERROR] in org.apache

[GitHub] [kylin] GuoNingNing commented on issue #624: Allow user-specified time format using real-time

GuoNingNing commented on issue #624: Allow user-specified time format using real-time URL: https://github.com/apache/kylin/pull/624#issuecomment-487814406 I am really sorry, this is the first time I submitted the source code to the open source community. I skipped checkstyle, and now this

[GitHub] [kylin] nichunen commented on issue #616: KYLIN-3965 JDBC fix.

nichunen commented on issue #616: KYLIN-3965 JDBC fix. URL: https://github.com/apache/kylin/pull/616#issuecomment-487849786 retest This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [kylin] lidongsjtu commented on issue #622: KYLIN-3893 Add validation for the encoding length and the type of encoding

lidongsjtu commented on issue #622: KYLIN-3893 Add validation for the encoding length and the type of encoding URL: https://github.com/apache/kylin/pull/622#issuecomment-487851029 retest This is an automated message from the

[GitHub] [kylin] lidongsjtu removed a comment on issue #622: KYLIN-3893 Add validation for the encoding length and the type of encoding

lidongsjtu removed a comment on issue #622: KYLIN-3893 Add validation for the encoding length and the type of encoding URL: https://github.com/apache/kylin/pull/622#issuecomment-487851029 retest This is an automated message f

[GitHub] [kylin] lidongsjtu commented on issue #622: KYLIN-3893 Add validation for the encoding length and the type of encoding

lidongsjtu commented on issue #622: KYLIN-3893 Add validation for the encoding length and the type of encoding URL: https://github.com/apache/kylin/pull/622#issuecomment-487853714 retest this please This is an automated messa

[GitHub] [kylin] nichunen commented on issue #616: KYLIN-3965 JDBC fix.

nichunen commented on issue #616: KYLIN-3965 JDBC fix. URL: https://github.com/apache/kylin/pull/616#issuecomment-487853759 retest this please This is an automated message from the Apache Git Service. To respond to the message

[GitHub] [kylin] nichunen removed a comment on issue #616: KYLIN-3965 JDBC fix.

nichunen removed a comment on issue #616: KYLIN-3965 JDBC fix. URL: https://github.com/apache/kylin/pull/616#issuecomment-487849786 retest This is an automated message from the Apache Git Service. To respond to the message, pl

[GitHub] [kylin] lidongsjtu removed a comment on issue #622: KYLIN-3893 Add validation for the encoding length and the type of encoding

lidongsjtu removed a comment on issue #622: KYLIN-3893 Add validation for the encoding length and the type of encoding URL: https://github.com/apache/kylin/pull/622#issuecomment-487853714 retest this please This is an automat

[GitHub] [kylin] coveralls commented on issue #624: Allow user-specified time format using real-time

coveralls commented on issue #624: Allow user-specified time format using real-time URL: https://github.com/apache/kylin/pull/624#issuecomment-487863142 ## Pull Request Test Coverage Report for [Build 4435](https://coveralls.io/builds/23101740) * **20** of **49** **(40.82%)** cha

[GitHub] [kylin] adoroszlai commented on issue #616: KYLIN-3965 JDBC fix.

adoroszlai commented on issue #616: KYLIN-3965 JDBC fix. URL: https://github.com/apache/kylin/pull/616#issuecomment-487863541 retest This is an automated message from the Apache Git Service. To respond to the message, please l

[GitHub] [kylin] codecov-io commented on issue #624: Allow user-specified time format using real-time

codecov-io commented on issue #624: Allow user-specified time format using real-time URL: https://github.com/apache/kylin/pull/624#issuecomment-487864284 # [Codecov](https://codecov.io/gh/apache/kylin/pull/624?src=pr&el=h1) Report > Merging [#624](https://codecov.io/gh/apache/kylin/pull/

[GitHub] [kylin] adoroszlai removed a comment on issue #616: KYLIN-3965 JDBC fix.

adoroszlai removed a comment on issue #616: KYLIN-3965 JDBC fix. URL: https://github.com/apache/kylin/pull/616#issuecomment-487863541 retest This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [kylin] Sidonet commented on issue #616: KYLIN-3965 JDBC fix.

Sidonet commented on issue #616: KYLIN-3965 JDBC fix. URL: https://github.com/apache/kylin/pull/616#issuecomment-487890922 > retest this please Hello, in my project i use build from this branch, due to DriverManager used in my project. How will i need to retest it? --

[GitHub] [kylin] nichunen commented on issue #616: KYLIN-3965 JDBC fix.

nichunen commented on issue #616: KYLIN-3965 JDBC fix. URL: https://github.com/apache/kylin/pull/616#issuecomment-487951198 > > retest this please > > Hello, in my project i use build from this branch, due to DriverManager used in my project. > How will i need to retest it?

[GitHub] [kylin] coveralls edited a comment on issue #616: KYLIN-3965 JDBC fix.

coveralls edited a comment on issue #616: KYLIN-3965 JDBC fix. URL: https://github.com/apache/kylin/pull/616#issuecomment-485441186 ## Pull Request Test Coverage Report for [Build 4436](https://coveralls.io/builds/23107154) * **0** of **0** changed or added relevant lines in **0**

[GitHub] [kylin] Sidonet commented on issue #616: KYLIN-3965 JDBC fix.

Sidonet commented on issue #616: KYLIN-3965 JDBC fix. URL: https://github.com/apache/kylin/pull/616#issuecomment-487966474 Hello Ni Chunen, Ok, i don't know about this feature. I think that the rat plugin expect license on jdbc/src/main/resources/META-INF/services/java.sql.Dri

[GitHub] [kylin] Sidonet edited a comment on issue #616: KYLIN-3965 JDBC fix.

Sidonet edited a comment on issue #616: KYLIN-3965 JDBC fix. URL: https://github.com/apache/kylin/pull/616#issuecomment-487966474 Hello Ni Chunen, Ok, i didn't know about this feature. I think that the rat plugin expect license on jdbc/src/main/resources/META-INF/services/java

[GitHub] [kylin] Sidonet edited a comment on issue #616: KYLIN-3965 JDBC fix.

Sidonet edited a comment on issue #616: KYLIN-3965 JDBC fix. URL: https://github.com/apache/kylin/pull/616#issuecomment-487966474 Hello Ni Chunen, Ok, i didn't know about this feature. I think that the rat plugin expect license on jdbc/src/main/resources/META-INF/services/java

[GitHub] [kylin] nichunen commented on issue #616: KYLIN-3965 JDBC fix.

nichunen commented on issue #616: KYLIN-3965 JDBC fix. URL: https://github.com/apache/kylin/pull/616#issuecomment-487994340 Hi @Sidonet I think you should add this file to the apache-rat-plugin's exclude list on pom.xml --

[GitHub] [kylin] coveralls edited a comment on issue #616: KYLIN-3965 JDBC fix.

coveralls edited a comment on issue #616: KYLIN-3965 JDBC fix. URL: https://github.com/apache/kylin/pull/616#issuecomment-485441186 ## Pull Request Test Coverage Report for [Build 4437](https://coveralls.io/builds/23110843) * **0** of **0** changed or added relevant lines in **0**

[GitHub] [kylin] codecov-io commented on issue #616: KYLIN-3965 JDBC fix.

codecov-io commented on issue #616: KYLIN-3965 JDBC fix. URL: https://github.com/apache/kylin/pull/616#issuecomment-488022349 # [Codecov](https://codecov.io/gh/apache/kylin/pull/616?src=pr&el=h1) Report > Merging [#616](https://codecov.io/gh/apache/kylin/pull/616?src=pr&el=desc) into [m

[GitHub] [kylin] TheRealHaui opened a new pull request #625: Added Unit Tests

TheRealHaui opened a new pull request #625: Added Unit Tests URL: https://github.com/apache/kylin/pull/625 Added Unit Tests to increase code coverage. This is an automated message from the Apache Git Service. To respond to the

[GitHub] [kylin] asfgit commented on issue #625: Added Unit Tests

asfgit commented on issue #625: Added Unit Tests URL: https://github.com/apache/kylin/pull/625#issuecomment-488059450 Can one of the admins verify this patch? This is an automated message from the Apache Git Service. To respon

[GitHub] [kylin] nichunen merged pull request #616: KYLIN-3965 JDBC fix.

nichunen merged pull request #616: KYLIN-3965 JDBC fix. URL: https://github.com/apache/kylin/pull/616 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the UR

[GitHub] [kylin] nichunen commented on issue #625: Added Unit Tests

nichunen commented on issue #625: Added Unit Tests URL: https://github.com/apache/kylin/pull/625#issuecomment-488186974 Hi @TheRealHaui The pr is not passed by style check, for "Using the '.*' form of import should be avoided" ---

[GitHub] [kylin] nichunen edited a comment on issue #625: Added Unit Tests

nichunen edited a comment on issue #625: Added Unit Tests URL: https://github.com/apache/kylin/pull/625#issuecomment-488186974 Hi @TheRealHaui The pr is not passed by style check, for "Using the '.*' form of import should be avoided" You can set your dev env with reference h

[GitHub] [kylin] TheRealHaui commented on issue #625: Added Unit Tests

TheRealHaui commented on issue #625: Added Unit Tests URL: https://github.com/apache/kylin/pull/625#issuecomment-488261919 Hi, thanks for the advice. I know. Already fixed it but oversaw that my IDE integrated a star import for JUnit assert statements on saving. Going to fix it

[GitHub] [kylin] coveralls commented on issue #625: Added Unit Tests

coveralls commented on issue #625: Added Unit Tests URL: https://github.com/apache/kylin/pull/625#issuecomment-488571436 ## Pull Request Test Coverage Report for [Build 4442](https://coveralls.io/builds/23141432) * **0** of **0** changed or added relevant lines in **0** files are

[GitHub] [kylin] codecov-io commented on issue #625: Added Unit Tests

codecov-io commented on issue #625: Added Unit Tests URL: https://github.com/apache/kylin/pull/625#issuecomment-488572495 # [Codecov](https://codecov.io/gh/apache/kylin/pull/625?src=pr&el=h1) Report > Merging [#625](https://codecov.io/gh/apache/kylin/pull/625?src=pr&el=desc) into [maste

[GitHub] [kylin] TheRealHaui commented on issue #625: Added Unit Tests

TheRealHaui commented on issue #625: Added Unit Tests URL: https://github.com/apache/kylin/pull/625#issuecomment-488573306 @nichunen, now it is passing. This is an automated message from the Apache Git Service. To respond

[GitHub] [kylin] yuexingri opened a new pull request #626: #3926, set sourceRecordCount when updating statistics

yuexingri opened a new pull request #626: #3926, set sourceRecordCount when updating statistics URL: https://github.com/apache/kylin/pull/626 rt This is an automated message from the Apache Git Service. To respond to the mess

[GitHub] [kylin] asfgit commented on issue #626: #3926, set sourceRecordCount when updating statistics

asfgit commented on issue #626: #3926, set sourceRecordCount when updating statistics URL: https://github.com/apache/kylin/pull/626#issuecomment-488926832 Can one of the admins verify this patch? This is an automated message

[GitHub] [kylin] coveralls commented on issue #626: #3926, set sourceRecordCount when updating statistics

coveralls commented on issue #626: #3926, set sourceRecordCount when updating statistics URL: https://github.com/apache/kylin/pull/626#issuecomment-488934371 ## Pull Request Test Coverage Report for [Build ](https://coveralls.io/builds/23164136) * **0** of **15** **(0.0%)** c

[GitHub] [kylin] nichunen merged pull request #625: Added Unit Tests

nichunen merged pull request #625: Added Unit Tests URL: https://github.com/apache/kylin/pull/625 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL ab

[GitHub] [kylin] yuexingri commented on issue #626: #3926, set sourceRecordCount when updating statistics

yuexingri commented on issue #626: #3926, set sourceRecordCount when updating statistics URL: https://github.com/apache/kylin/pull/626#issuecomment-489290692 retest this please This is an automated message from the Apache Git

[GitHub] [kylin] shaofengshi commented on issue #626: #3926, set sourceRecordCount when updating statistics

shaofengshi commented on issue #626: #3926, set sourceRecordCount when updating statistics URL: https://github.com/apache/kylin/pull/626#issuecomment-489326878 duplicated with https://github.com/apache/kylin/pull/579 This is

[GitHub] [kylin] Qsimple opened a new pull request #627: replace "\n" by System.lineseperator to support run test case

Qsimple opened a new pull request #627: replace "\n" by System.lineseperator to support run test case URL: https://github.com/apache/kylin/pull/627 replace "\n" by System.lineseperator to support run test case successfully on windows platform -

[GitHub] [kylin] asfgit commented on issue #627: replace "\n" by System.lineseperator to support run test case

asfgit commented on issue #627: replace "\n" by System.lineseperator to support run test case URL: https://github.com/apache/kylin/pull/627#issuecomment-489336303 Can one of the admins verify this patch? This is an automated

[GitHub] [kylin] coveralls commented on issue #627: replace "\n" by System.lineseperator to support run test case

coveralls commented on issue #627: replace "\n" by System.lineseperator to support run test case URL: https://github.com/apache/kylin/pull/627#issuecomment-489338788 ## Pull Request Test Coverage Report for [Build 4446](https://coveralls.io/builds/23189439) * **0** of **0** change

[GitHub] [kylin] codecov-io commented on issue #626: #3926, set sourceRecordCount when updating statistics

codecov-io commented on issue #626: #3926, set sourceRecordCount when updating statistics URL: https://github.com/apache/kylin/pull/626#issuecomment-489379799 # [Codecov](https://codecov.io/gh/apache/kylin/pull/626?src=pr&el=h1) Report > Merging [#626](https://codecov.io/gh/apache/kylin/

[GitHub] [kylin] coveralls edited a comment on issue #626: #3926, set sourceRecordCount when updating statistics

coveralls edited a comment on issue #626: #3926, set sourceRecordCount when updating statistics URL: https://github.com/apache/kylin/pull/626#issuecomment-488934371 ## Pull Request Test Coverage Report for [Build 4447](https://coveralls.io/builds/23193336) * **0** of **15** **(0.0

[GitHub] [kylin] asfgit commented on issue #628: KYLIN-3980 Cube planner cuboid id error when dimensions too many

asfgit commented on issue #628: KYLIN-3980 Cube planner cuboid id error when dimensions too many URL: https://github.com/apache/kylin/pull/628#issuecomment-489385506 Can one of the admins verify this patch? This is an automat

[GitHub] [kylin] sanjulian opened a new pull request #628: KYLIN-3980 Cube planner cuboid id error when dimensions too many

sanjulian opened a new pull request #628: KYLIN-3980 Cube planner cuboid id error when dimensions too many URL: https://github.com/apache/kylin/pull/628 This is an automated message from the Apache Git Service. To respond to

[GitHub] [kylin] codecov-io commented on issue #628: KYLIN-3980 Cube planner cuboid id error when dimensions too many

codecov-io commented on issue #628: KYLIN-3980 Cube planner cuboid id error when dimensions too many URL: https://github.com/apache/kylin/pull/628#issuecomment-489387324 # [Codecov](https://codecov.io/gh/apache/kylin/pull/628?src=pr&el=h1) Report > Merging [#628](https://codecov.io/gh/ap

[GitHub] [kylin] coveralls commented on issue #628: KYLIN-3980 Cube planner cuboid id error when dimensions too many

coveralls commented on issue #628: KYLIN-3980 Cube planner cuboid id error when dimensions too many URL: https://github.com/apache/kylin/pull/628#issuecomment-489387762 ## Pull Request Test Coverage Report for [Build 4449](https://coveralls.io/builds/23194069) * **0** of **4** **(

[GitHub] [kylin] nichunen merged pull request #627: replace "\n" by System.lineseperator to support run test case

nichunen merged pull request #627: replace "\n" by System.lineseperator to support run test case URL: https://github.com/apache/kylin/pull/627 This is an automated message from the Apache Git Service. To respond to the messa

[GitHub] [kylin] nichunen merged pull request #582: Read password from file for Beeline

nichunen merged pull request #582: Read password from file for Beeline URL: https://github.com/apache/kylin/pull/582 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [kylin] asfgit commented on issue #629: KYLIN-3983 Add extra metadata for measure. Add test case and example/…

asfgit commented on issue #629: KYLIN-3983 Add extra metadata for measure. Add test case and example/… URL: https://github.com/apache/kylin/pull/629#issuecomment-489419990 Can one of the admins verify this patch? This is an a

[GitHub] [kylin] Qsimple opened a new pull request #629: KYLIN-3983 Add extra metadata for measure. Add test case and example/…

Qsimple opened a new pull request #629: KYLIN-3983 Add extra metadata for measure. Add test case and example/… URL: https://github.com/apache/kylin/pull/629 …xample/test_case_data for MeasureManager UT pass IT pass

[GitHub] [kylin] coveralls commented on issue #619: #KYLIN-3977

coveralls commented on issue #619: #KYLIN-3977 URL: https://github.com/apache/kylin/pull/619#issuecomment-489493944 ## Pull Request Test Coverage Report for [Build 4454](https://coveralls.io/builds/23203594) * **161** of **357** **(45.1%)** changed or added relevant lines in **13

[GitHub] [kylin] codecov-io commented on issue #619: #KYLIN-3977

codecov-io commented on issue #619: #KYLIN-3977 URL: https://github.com/apache/kylin/pull/619#issuecomment-489494288 # [Codecov](https://codecov.io/gh/apache/kylin/pull/619?src=pr&el=h1) Report > Merging [#619](https://codecov.io/gh/apache/kylin/pull/619?src=pr&el=desc) into [master](ht

[GitHub] [kylin] coveralls edited a comment on issue #619: #KYLIN-3977

coveralls edited a comment on issue #619: #KYLIN-3977 URL: https://github.com/apache/kylin/pull/619#issuecomment-489493944 ## Pull Request Test Coverage Report for [Build 4456](https://coveralls.io/builds/23206272) * **161** of **357** **(45.1%)** changed or added relevant lines i

[GitHub] [kylin] codecov-io edited a comment on issue #619: #KYLIN-3977

codecov-io edited a comment on issue #619: #KYLIN-3977 URL: https://github.com/apache/kylin/pull/619#issuecomment-489494288 # [Codecov](https://codecov.io/gh/apache/kylin/pull/619?src=pr&el=h1) Report > Merging [#619](https://codecov.io/gh/apache/kylin/pull/619?src=pr&el=desc) into [mas

[GitHub] [kylin] ZhengshuaiPENG opened a new pull request #630: #KYLIN-3913 Remove getAllOutputs api in ExecutableManager to avoid OOM for large metadata

ZhengshuaiPENG opened a new pull request #630: #KYLIN-3913 Remove getAllOutputs api in ExecutableManager to avoid OOM for large metadata URL: https://github.com/apache/kylin/pull/630 see https://issues.apache.org/jira/browse/KYLIN-3913 and pr from @lshmouse https://github.com/apache/kyli

[GitHub] [kylin] asfgit commented on issue #630: #KYLIN-3913 Remove getAllOutputs api in ExecutableManager to avoid OOM for large metadata

asfgit commented on issue #630: #KYLIN-3913 Remove getAllOutputs api in ExecutableManager to avoid OOM for large metadata URL: https://github.com/apache/kylin/pull/630#issuecomment-489560125 Can one of the admins verify this patch? --

[GitHub] [kylin] coveralls commented on issue #630: #KYLIN-3913 Remove getAllOutputs api in ExecutableManager to avoid OOM for large metadata

coveralls commented on issue #630: #KYLIN-3913 Remove getAllOutputs api in ExecutableManager to avoid OOM for large metadata URL: https://github.com/apache/kylin/pull/630#issuecomment-489581926 ## Pull Request Test Coverage Report for [Build 4458](https://coveralls.io/builds/23209567)

[GitHub] [kylin] nichunen commented on issue #557: KYLIN-3913: Remove getAllOutputs api in ExecutableManager to avoid OOM for large metadata

nichunen commented on issue #557: KYLIN-3913: Remove getAllOutputs api in ExecutableManager to avoid OOM for large metadata URL: https://github.com/apache/kylin/pull/557#issuecomment-489612225 It will be done in https://github.com/apache/kylin/pull/630, the commit in this pr will be kept t

[GitHub] [kylin] nichunen closed pull request #557: KYLIN-3913: Remove getAllOutputs api in ExecutableManager to avoid OOM for large metadata

nichunen closed pull request #557: KYLIN-3913: Remove getAllOutputs api in ExecutableManager to avoid OOM for large metadata URL: https://github.com/apache/kylin/pull/557 This is an automated message from the Apache Git Serv

[GitHub] [kylin] TheRealHaui opened a new pull request #631: Add additional unit tests

TheRealHaui opened a new pull request #631: Add additional unit tests URL: https://github.com/apache/kylin/pull/631 Added Unit Tests to increase code coverage. This is an automated message from the Apache Git Service. To respo

[GitHub] [kylin] asfgit commented on issue #631: Add additional unit tests

asfgit commented on issue #631: Add additional unit tests URL: https://github.com/apache/kylin/pull/631#issuecomment-489613014 Can one of the admins verify this patch? This is an automated message from the Apache Git Service.

[GitHub] [kylin] coveralls edited a comment on issue #533: kylin-3812 optimize the child CompareTupleFilter in a CompareTupleFilter

coveralls edited a comment on issue #533: kylin-3812 optimize the child CompareTupleFilter in a CompareTupleFilter URL: https://github.com/apache/kylin/pull/533#issuecomment-474183735 ## Pull Request Test Coverage Report for [Build 4459](https://coveralls.io/builds/23212111) * **2**

[GitHub] [kylin] nichunen commented on issue #618: KYLIN-3969 Fix Beeline meta data parser on partition information

nichunen commented on issue #618: KYLIN-3969 Fix Beeline meta data parser on partition information URL: https://github.com/apache/kylin/pull/618#issuecomment-489653473 @sickate Hi, fix for the compatibility with hive 3 is done in branch master-hadoop3.1, would you please re-submit a pr? -

[GitHub] [kylin] coveralls edited a comment on issue #533: kylin-3812 optimize the child CompareTupleFilter in a CompareTupleFilter

coveralls edited a comment on issue #533: kylin-3812 optimize the child CompareTupleFilter in a CompareTupleFilter URL: https://github.com/apache/kylin/pull/533#issuecomment-474183735 ## Pull Request Test Coverage Report for [Build 4462](https://coveralls.io/builds/23226363) * **2**

[GitHub] [kylin] nichunen closed pull request #626: #3926, set sourceRecordCount when updating statistics

nichunen closed pull request #626: #3926, set sourceRecordCount when updating statistics URL: https://github.com/apache/kylin/pull/626 This is an automated message from the Apache Git Service. To respond to the message, plea

[GitHub] [kylin] nichunen commented on issue #626: #3926, set sourceRecordCount when updating statistics

nichunen commented on issue #626: #3926, set sourceRecordCount when updating statistics URL: https://github.com/apache/kylin/pull/626#issuecomment-489886972 duplicated with #579, close it This is an automated message from the

[GitHub] [kylin] asfgit commented on issue #632: KYLIN-3994: StorageCleanupJob may delete cube id data of new built segment because of cube cache in CubeManager

asfgit commented on issue #632: KYLIN-3994: StorageCleanupJob may delete cube id data of new built segment because of cube cache in CubeManager URL: https://github.com/apache/kylin/pull/632#issuecomment-489923398 Can one of the admins verify this patch?

[GitHub] [kylin] lshmouse opened a new pull request #632: KYLIN-3994: StorageCleanupJob may delete cube id data of new built segment because of cube cache in CubeManager

lshmouse opened a new pull request #632: KYLIN-3994: StorageCleanupJob may delete cube id data of new built segment because of cube cache in CubeManager URL: https://github.com/apache/kylin/pull/632 See: https://issues.apache.org/jira/browse/KYLIN-3994 -

[GitHub] [kylin] coveralls edited a comment on issue #579: KYLIN-3926 set sourceRecordCount when updating statistics

coveralls edited a comment on issue #579: KYLIN-3926 set sourceRecordCount when updating statistics URL: https://github.com/apache/kylin/pull/579#issuecomment-479439737 ## Pull Request Test Coverage Report for [Build 4464](https://coveralls.io/builds/23228361) * **0** of **51** **

[GitHub] [kylin] lshmouse closed pull request #632: KYLIN-3994: StorageCleanupJob may delete cube id data of new built segment because of cube cache in CubeManager

lshmouse closed pull request #632: KYLIN-3994: StorageCleanupJob may delete cube id data of new built segment because of cube cache in CubeManager URL: https://github.com/apache/kylin/pull/632 This is an automated message fr

[GitHub] [kylin] codecov-io edited a comment on issue #579: KYLIN-3926 set sourceRecordCount when updating statistics

codecov-io edited a comment on issue #579: KYLIN-3926 set sourceRecordCount when updating statistics URL: https://github.com/apache/kylin/pull/579#issuecomment-479440908 # [Codecov](https://codecov.io/gh/apache/kylin/pull/579?src=pr&el=h1) Report > Merging [#579](https://codecov.io/gh/ap

[GitHub] [kylin] asfgit commented on issue #633: KYLIN-3994: StorageCleanupJob may delete cube id data of new built segment because of cube cache in CubeManager

asfgit commented on issue #633: KYLIN-3994: StorageCleanupJob may delete cube id data of new built segment because of cube cache in CubeManager URL: https://github.com/apache/kylin/pull/633#issuecomment-489928543 Can one of the admins verify this patch?

[GitHub] [kylin] lshmouse opened a new pull request #633: KYLIN-3994: StorageCleanupJob may delete cube id data of new built segment because of cube cache in CubeManager

lshmouse opened a new pull request #633: KYLIN-3994: StorageCleanupJob may delete cube id data of new built segment because of cube cache in CubeManager URL: https://github.com/apache/kylin/pull/633 See: https://issues.apache.org/jira/browse/KYLIN-3994 -

[GitHub] [kylin] coveralls commented on issue #632: KYLIN-3994: StorageCleanupJob may delete cube id data of new built segment because of cube cache in CubeManager

coveralls commented on issue #632: KYLIN-3994: StorageCleanupJob may delete cube id data of new built segment because of cube cache in CubeManager URL: https://github.com/apache/kylin/pull/632#issuecomment-489929591 ## Pull Request Test Coverage Report for [Build 4465](https://coveralls.io

[GitHub] [kylin] coveralls commented on issue #633: KYLIN-3994: StorageCleanupJob may delete cube id data of new built segment because of cube cache in CubeManager

coveralls commented on issue #633: KYLIN-3994: StorageCleanupJob may delete cube id data of new built segment because of cube cache in CubeManager URL: https://github.com/apache/kylin/pull/633#issuecomment-489959857 ## Pull Request Test Coverage Report for [Build 4466](https://coveralls.io

[GitHub] [kylin] chenzhx opened a new pull request #634: KYLIN-3995 config data type may make oom

chenzhx opened a new pull request #634: KYLIN-3995 config data type may make oom URL: https://github.com/apache/kylin/pull/634 jira : https://issues.apache.org/jira/browse/KYLIN-3995 This is an automated message from the Apach

[GitHub] [kylin] asfgit commented on issue #634: KYLIN-3995 config data type may make oom

asfgit commented on issue #634: KYLIN-3995 config data type may make oom URL: https://github.com/apache/kylin/pull/634#issuecomment-489980857 Can one of the admins verify this patch? This is an automated message from the Apach

[GitHub] [kylin] coveralls commented on issue #634: KYLIN-3995 config data type may make oom

coveralls commented on issue #634: KYLIN-3995 config data type may make oom URL: https://github.com/apache/kylin/pull/634#issuecomment-489989213 ## Pull Request Test Coverage Report for [Build 4467](https://coveralls.io/builds/23230429) * **2** of **2** **(100.0%)** changed or add

[GitHub] [kylin] nichunen merged pull request #634: KYLIN-3995 config data type may make oom

nichunen merged pull request #634: KYLIN-3995 config data type may make oom URL: https://github.com/apache/kylin/pull/634 This is an automated message from the Apache Git Service. To respond to the message, please log on to G

[GitHub] [kylin] coveralls edited a comment on issue #533: kylin-3812 optimize the child CompareTupleFilter in a CompareTupleFilter

coveralls edited a comment on issue #533: kylin-3812 optimize the child CompareTupleFilter in a CompareTupleFilter URL: https://github.com/apache/kylin/pull/533#issuecomment-474183735 ## Pull Request Test Coverage Report for [Build 4468](https://coveralls.io/builds/23231285) * **2**

[GitHub] [kylin] codecov-io commented on issue #533: kylin-3812 optimize the child CompareTupleFilter in a CompareTupleFilter

codecov-io commented on issue #533: kylin-3812 optimize the child CompareTupleFilter in a CompareTupleFilter URL: https://github.com/apache/kylin/pull/533#issuecomment-490004354 # [Codecov](https://codecov.io/gh/apache/kylin/pull/533?src=pr&el=h1) Report > Merging [#533](https://codecov.

[GitHub] [kylin] nichunen commented on a change in pull request #579: KYLIN-3926 set sourceRecordCount when updating statistics

nichunen commented on a change in pull request #579: KYLIN-3926 set sourceRecordCount when updating statistics URL: https://github.com/apache/kylin/pull/579#discussion_r273989590 ## File path: engine-mr/src/main/java/org/apache/kylin/engine/mr/steps/MergeStatisticsStep.java ##

<    6   7   8   9   10   11   12   >