shaofengshi commented on issue #534: KYLIN-3893: Cube build failed for wrong
row key column description
URL: https://github.com/apache/kylin/pull/534#issuecomment-475919897
This is a kind of input validation, the best place is the first time when
the cube is saved, in
asfgit commented on issue #554: Minor, change travis ci log level to WARN
URL: https://github.com/apache/kylin/pull/554#issuecomment-476945942
Can one of the admins verify this patch?
This is an automated message from the
asfgit commented on issue #554: Minor, change travis ci log level to WARN
URL: https://github.com/apache/kylin/pull/554#issuecomment-476945945
Can one of the admins verify this patch?
This is an automated message from the
vaful commented on a change in pull request #533: kylin-3812 optimize the child
CompareTupleFilter in a CompareTupleFilter
URL: https://github.com/apache/kylin/pull/533#discussion_r269391463
##
File path:
codecov-io commented on issue #554: Minor, change travis ci log level to WARN
URL: https://github.com/apache/kylin/pull/554#issuecomment-476954158
# [Codecov](https://codecov.io/gh/apache/kylin/pull/554?src=pr=h1) Report
> :exclamation: No coverage uploaded for pull request base
vaful commented on a change in pull request #533: kylin-3812 optimize the child
CompareTupleFilter in a CompareTupleFilter
URL: https://github.com/apache/kylin/pull/533#discussion_r269391576
##
File path:
shaofengshi merged pull request #549: KYLIN-3905 Enable shrunken dictionary
default
URL: https://github.com/apache/kylin/pull/549
This is an automated message from the Apache Git Service.
To respond to the message, please
shaofengshi merged pull request #538: KYLIN-3898 Cube level properties are
ineffective in the some build steps
URL: https://github.com/apache/kylin/pull/538
This is an automated message from the Apache Git Service.
To
shaofengshi merged pull request #550: KYLIN-3909: kylin job failed for
MappeableRunContainer is not registered
URL: https://github.com/apache/kylin/pull/550
This is an automated message from the Apache Git Service.
To
shaofengshi merged pull request #545: KYLIN-3902 fix JoinDesc in case of same
fact column with multiple loo…
URL: https://github.com/apache/kylin/pull/545
This is an automated message from the Apache Git Service.
To respond
shaofengshi commented on issue #554: Minor, change travis ci log level to WARN
URL: https://github.com/apache/kylin/pull/554#issuecomment-476954487
+1
This is an automated message from the Apache Git Service.
To respond to
shaofengshi merged pull request #554: Minor, change travis ci log level to WARN
URL: https://github.com/apache/kylin/pull/554
This is an automated message from the Apache Git Service.
To respond to the message, please log on
codecov-io commented on issue #555: KYLIN-3911: Check if HBase table is enabled
before diabling table in DeployCoprocessorCLI
URL: https://github.com/apache/kylin/pull/555#issuecomment-476983427
# [Codecov](https://codecov.io/gh/apache/kylin/pull/555?src=pr=h1) Report
> :exclamation: No
asfgit commented on issue #556: KYLIN-3912: Support cube level mapreduuce queue
config for BeelineHiveClient
URL: https://github.com/apache/kylin/pull/556#issuecomment-476985750
Can one of the admins verify this patch?
This
asfgit commented on issue #556: KYLIN-3912: Support cube level mapreduuce queue
config for BeelineHiveClient
URL: https://github.com/apache/kylin/pull/556#issuecomment-476985751
Can one of the admins verify this patch?
This
lshmouse opened a new pull request #556: KYLIN-3912: Support cube level
mapreduuce queue config for BeelineHiveClient
URL: https://github.com/apache/kylin/pull/556
See: https://issues.apache.org/jira/browse/KYLIN-3912
Add hive confs in kylin config to hive jdbc connection
asfgit 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-476990149
Can one of the admins verify this patch?
lshmouse opened a new pull request #557: KYLIN-3913: Remove getAllOutputs api
in ExecutableManager to avoid OOM for large metadata
URL: https://github.com/apache/kylin/pull/557
See: https://issues.apache.org/jira/browse/KYLIN-3913
asfgit 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-476990148
Can one of the admins verify this patch?
codecov-io commented on issue #556: KYLIN-3912: Support cube level mapreduuce
queue config for BeelineHiveClient
URL: https://github.com/apache/kylin/pull/556#issuecomment-476994095
# [Codecov](https://codecov.io/gh/apache/kylin/pull/556?src=pr=h1) Report
> :exclamation: No coverage
codecov-io 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-47690
# [Codecov](https://codecov.io/gh/apache/kylin/pull/557?src=pr=h1) Report
> :exclamation: No
shaofengshi closed pull request #471: KYLIN-3815 Unexpected behavior when
joinning streaming table and hive table
URL: https://github.com/apache/kylin/pull/471
This is an automated message from the Apache Git Service.
To
shaofengshi commented on issue #471: KYLIN-3815 Unexpected behavior when
joinning streaming table and hive table
URL: https://github.com/apache/kylin/pull/471#issuecomment-465005666
Looks good; merged with patch, so close this PR.
codecov-io commented on issue #479: KYLIN-3826 MergeCuboidJob only uploads
necessary segment's dictionary
URL: https://github.com/apache/kylin/pull/479#issuecomment-466723919
# [Codecov](https://codecov.io/gh/apache/kylin/pull/479?src=pr=h1) Report
> :exclamation: No coverage uploaded
asfgit commented on issue #480: Update documents
URL: https://github.com/apache/kylin/pull/480#issuecomment-466725279
Can one of the admins verify this patch?
This is an automated message from the Apache Git Service.
To
coveralls commented on issue #479: KYLIN-3826 MergeCuboidJob only uploads
necessary segment's dictionary
URL: https://github.com/apache/kylin/pull/479#issuecomment-466745690
## Pull Request Test Coverage Report for [Build
4158](https://coveralls.io/builds/21812504)
* **0** of **3**
GinaZhai opened a new pull request #480: Update documents
URL: https://github.com/apache/kylin/pull/480
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
shaofengshi merged pull request #479: KYLIN-3826 MergeCuboidJob only uploads
necessary segment's dictionary
URL: https://github.com/apache/kylin/pull/479
This is an automated message from the Apache Git Service.
To respond
asfgit commented on issue #479: KYLIN-3826 MergeCuboidJob only uploads
necessary segment's dictionary
URL: https://github.com/apache/kylin/pull/479#issuecomment-466720183
Can one of the admins verify this patch?
This is an
shaofengshi opened a new pull request #479: KYLIN-3826 MergeCuboidJob only
uploads necessary segment's dictionary
URL: https://github.com/apache/kylin/pull/479
This is an automated message from the Apache Git Service.
To
asfgit commented on issue #485: KYLIN-3829 Chinese document on the kylin
official site about sample c…
URL: https://github.com/apache/kylin/pull/485#issuecomment-467341431
Can one of the admins verify this patch?
This is an
GinaZhai opened a new pull request #485: KYLIN-3829 Chinese document on the
kylin official site about sample c…
URL: https://github.com/apache/kylin/pull/485
…ube got the sql wrong
This is an automated message from the
asfgit commented on issue #486: KYLIN-3833 Avoid OOM.
URL: https://github.com/apache/kylin/pull/486#issuecomment-467494240
Can one of the admins verify this patch?
This is an automated message from the Apache Git Service.
To
Sidonet opened a new pull request #486: KYLIN-3833 Avoid OOM.
URL: https://github.com/apache/kylin/pull/486
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use
Aaron commented on issue #484: KYLIN-3820 Add a curator-based scheduler
URL: https://github.com/apache/kylin/pull/484#issuecomment-467484764
Please hold on a moment.
This is an automated message from the Apache Git
Aaron commented on issue #484: KYLIN-3820 Add a curator-based scheduler
URL: https://github.com/apache/kylin/pull/484#issuecomment-467484764
Please hold on a moment.
This is an automated message from the Apache Git
Aaron edited a comment on issue #484: KYLIN-3820 Add a curator-based
scheduler
URL: https://github.com/apache/kylin/pull/484#issuecomment-467484764
Please wait for a moment.
This is an automated message from the Apache
Aaron edited a comment on issue #484: KYLIN-3820 Add a curator-based
scheduler
URL: https://github.com/apache/kylin/pull/484#issuecomment-467484764
Please wait for a moment.
This is an automated message from the Apache
Sidonet opened a new pull request #487: KYLIN-3714 Register kryo for spark
spilling process.
URL: https://github.com/apache/kylin/pull/487
After patch was applied in my env, i test it.
In my test case, i set properties
kylin.engine.spark-conf.spark.driver.memory=2G
asfgit commented on issue #487: KYLIN-3714 Register kryo for spark spilling
process.
URL: https://github.com/apache/kylin/pull/487#issuecomment-467496008
Can one of the admins verify this patch?
This is an automated message
shaofengshi merged pull request #483: KYLIN-3816 Make CI cover streaming table
join case
URL: https://github.com/apache/kylin/pull/483
This is an automated message from the Apache Git Service.
To respond to the message,
shaofengshi merged pull request #483: KYLIN-3816 Make CI cover streaming table
join case
URL: https://github.com/apache/kylin/pull/483
This is an automated message from the Apache Git Service.
To respond to the message,
shaofengshi merged pull request #485: KYLIN-3829 Chinese document on the kylin
official site about sample c…
URL: https://github.com/apache/kylin/pull/485
This is an automated message from the Apache Git Service.
To respond
shaofengshi merged pull request #485: KYLIN-3829 Chinese document on the kylin
official site about sample c…
URL: https://github.com/apache/kylin/pull/485
This is an automated message from the Apache Git Service.
To respond
Sidonet opened a new pull request #494: KYLIN-3727 Check if where is no
directories, then finish job successf…
URL: https://github.com/apache/kylin/pull/494
…ully.
Input path for HBase load consists
_SUCCESS
F1
F2
etc...
So, check if no folders exist, then skip
codecov-io commented on issue #494: KYLIN-3727 Check if where is no
directories, then finish job successf…
URL: https://github.com/apache/kylin/pull/494#issuecomment-468257977
# [Codecov](https://codecov.io/gh/apache/kylin/pull/494?src=pr=h1) Report
> :exclamation: No coverage uploaded
shaofengshi merged pull request #493: KYLIN-3836 fix Kylin StringUtil.join()
may cause NPE if iterator is e…
URL: https://github.com/apache/kylin/pull/493
This is an automated message from the Apache Git Service.
To respond
asfgit commented on issue #494: KYLIN-3727 Check if where is no directories,
then finish job successf…
URL: https://github.com/apache/kylin/pull/494#issuecomment-468250666
Can one of the admins verify this patch?
This is an
allenma commented on a change in pull request #481: KYLIN-3821 Add consume lag
stats
URL: https://github.com/apache/kylin/pull/481#discussion_r260692914
##
File path:
stream-source-kafka/src/main/java/org/apache/kylin/stream/source/kafka/consumer/KafkaConnector.java
##
allenma commented on a change in pull request #481: KYLIN-3821 Add consume lag
stats
URL: https://github.com/apache/kylin/pull/481#discussion_r260695771
##
File path:
stream-core/src/main/java/org/apache/kylin/stream/core/consumer/IStreamingConnector.java
##
@@ -1,46
asfgit commented on issue #490: Added small Unit Tests
URL: https://github.com/apache/kylin/pull/490#issuecomment-467932233
Can one of the admins verify this patch?
This is an automated message from the Apache Git Service.
To
TheRealHaui opened a new pull request #490: Added small Unit Tests
URL: https://github.com/apache/kylin/pull/490
Added small Unit Tests
This is an automated message from the Apache Git Service.
To respond to the message,
Wayne1c commented on issue #489: KYLIN-2620 Make the condition stricter to
answer query with topN
URL: https://github.com/apache/kylin/pull/489#issuecomment-468513152
CI passed
This is an automated message from the Apache
coveralls edited a comment on issue #489: KYLIN-2620 Make the condition
stricter to answer query with topN
URL: https://github.com/apache/kylin/pull/489#issuecomment-467762777
## Pull Request Test Coverage Report for [Build
4178](https://coveralls.io/builds/21894643)
* **0** of
codecov-io commented on issue #489: KYLIN-2620 Make the condition stricter to
answer query with topN
URL: https://github.com/apache/kylin/pull/489#issuecomment-468638893
# [Codecov](https://codecov.io/gh/apache/kylin/pull/489?src=pr=h1) Report
> :exclamation: No coverage uploaded for
shaofengshi merged pull request #494: KYLIN-3727 Check if where is no
directories, then finish job successf…
URL: https://github.com/apache/kylin/pull/494
This is an automated message from the Apache Git Service.
To respond
coveralls commented on issue #492: KYLIN-3835 [Defective
TableSchemaUpdateChecker]
URL: https://github.com/apache/kylin/pull/492#issuecomment-468811366
## Pull Request Test Coverage Report for [Build
4181](https://coveralls.io/builds/21897019)
* **0** of **75** **(0.0%)**
codecov-io commented on issue #496: KYLIN-3838 Fix retry mechanism is invalid
when build with spark
URL: https://github.com/apache/kylin/pull/496#issuecomment-469108767
# [Codecov](https://codecov.io/gh/apache/kylin/pull/496?src=pr=h1) Report
> :exclamation: No coverage uploaded for
asfgit commented on issue #497: KYLIN-3795 Submit Spark jobs via Apache Livy
URL: https://github.com/apache/kylin/pull/497#issuecomment-469111500
Can one of the admins verify this patch?
This is an automated message from the
javalife0312 opened a new pull request #497: KYLIN-3795 Submit Spark jobs via
Apache Livy
URL: https://github.com/apache/kylin/pull/497
KYLIN-3795 Submit Spark jobs via Apache Livy
This is an automated message from the
javalife0312 closed pull request #475: kylin - livy
URL: https://github.com/apache/kylin/pull/475
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
codecov-io edited a comment on issue #491: KYLIN-3820 KYLIN-3834 Add a
curator-based scheduler
URL: https://github.com/apache/kylin/pull/491#issuecomment-469147125
# [Codecov](https://codecov.io/gh/apache/kylin/pull/491?src=pr=h1) Report
> :exclamation: No coverage uploaded for pull
Qsimple commented on issue #492: KYLIN-3835 [Defective
TableSchemaUpdateChecker]
URL: https://github.com/apache/kylin/pull/492#issuecomment-469103057
Hi team:
Am I modify too many code to merge this pull request?
asfgit commented on issue #496: KYLIN-3838 Fix retry mechanism is invalid when
build with spark
URL: https://github.com/apache/kylin/pull/496#issuecomment-469105008
Can one of the admins verify this patch?
This is an
Wayne1c opened a new pull request #496: KYLIN-3838 Fix retry mechanism is
invalid when build with spark
URL: https://github.com/apache/kylin/pull/496
This is an automated message from the Apache Git Service.
To respond to
coveralls commented on issue #494: KYLIN-3727 Check if where is no directories,
then finish job successf…
URL: https://github.com/apache/kylin/pull/494#issuecomment-468897839
## Pull Request Test Coverage Report for [Build
4184](https://coveralls.io/builds/21903449)
* **0** of
shaofengshi merged pull request #491: KYLIN-3820 KYLIN-3834 Add a curator-based
scheduler
URL: https://github.com/apache/kylin/pull/491
This is an automated message from the Apache Git Service.
To respond to the message,
shaofengshi merged pull request #496: KYLIN-3838 Fix retry mechanism is invalid
when build with spark
URL: https://github.com/apache/kylin/pull/496
This is an automated message from the Apache Git Service.
To respond to the
shaofengshi commented on issue #492: KYLIN-3835 [Defective
TableSchemaUpdateChecker]
URL: https://github.com/apache/kylin/pull/492#issuecomment-469229372
The change looks good! Merged with git commit (in order to exclude the merge
commit from the pr).
Thank you Yuzhang for making
shaofengshi closed pull request #436: KYLIN-3716 FastThreadLocal replaces
ThreadLocal
URL: https://github.com/apache/kylin/pull/436
This is an automated message from the Apache Git Service.
To respond to the message, please
shaofengshi commented on issue #436: KYLIN-3716 FastThreadLocal replaces
ThreadLocal
URL: https://github.com/apache/kylin/pull/436#issuecomment-469224459
Hi Nicholas, I merge the change by merging the two commits into one, and
modified the commit message to follow Apache rule. Thanks for
shaofengshi commented on issue #490: Added small Unit Tests
URL: https://github.com/apache/kylin/pull/490#issuecomment-469230529
Hi Michael, thanks for contributing to Kylin! The change seems not passing
the unit test, could you please take a look? Just let me know if you need
support.
allenma merged pull request #481: KYLIN-3821 Add consume lag stats
URL: https://github.com/apache/kylin/pull/481
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
shaofengshi merged pull request #497: KYLIN-3795 Submit Spark jobs via Apache
Livy
URL: https://github.com/apache/kylin/pull/497
This is an automated message from the Apache Git Service.
To respond to the message, please
shaofengshi merged pull request #489: KYLIN-2620 Make the condition stricter to
answer query with topN
URL: https://github.com/apache/kylin/pull/489
This is an automated message from the Apache Git Service.
To respond to
shaofengshi closed pull request #492: KYLIN-3835 [Defective
TableSchemaUpdateChecker]
URL: https://github.com/apache/kylin/pull/492
This is an automated message from the Apache Git Service.
To respond to the message,
coveralls edited a comment on issue #489: KYLIN-2620 Make the condition
stricter to answer query with topN
URL: https://github.com/apache/kylin/pull/489#issuecomment-467762777
## Pull Request Test Coverage Report for [Build
4186](https://coveralls.io/builds/21926359)
* **0** of
asfgit commented on issue #498: Realtime olap release
URL: https://github.com/apache/kylin/pull/498#issuecomment-469236457
Can one of the admins verify this patch?
This is an automated message from the Apache Git Service.
To
hit-lacus opened a new pull request #498: Realtime olap release
URL: https://github.com/apache/kylin/pull/498
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
TheRealHaui commented on issue #490: Added small Unit Tests
URL: https://github.com/apache/kylin/pull/490#issuecomment-469279196
@shaofengshi,
thanks for your kind reply!
Oversaw that your build servers complained.
The problem was a checkstyle rule.
Removed .* imports from
codecov-io commented on issue #497: KYLIN-3795 Submit Spark jobs via Apache Livy
URL: https://github.com/apache/kylin/pull/497#issuecomment-469115848
# [Codecov](https://codecov.io/gh/apache/kylin/pull/497?src=pr=h1) Report
> :exclamation: No coverage uploaded for pull request base
codecov-io commented on issue #491: KYLIN-3820 KYLIN-3834 Add a curator-based
scheduler
URL: https://github.com/apache/kylin/pull/491#issuecomment-469147125
# [Codecov](https://codecov.io/gh/apache/kylin/pull/491?src=pr=h1) Report
> :exclamation: No coverage uploaded for pull request
coveralls commented on issue #473: add Apache find-spark-home fallback script
URL: https://github.com/apache/kylin/pull/473#issuecomment-465823842
## Pull Request Test Coverage Report for [Build
4147](https://coveralls.io/builds/21747064)
* **0** of **0** changed or added relevant
coveralls commented on issue #474: KYLIN-3537
URL: https://github.com/apache/kylin/pull/474#issuecomment-465853240
## Pull Request Test Coverage Report for [Build
4148](https://coveralls.io/builds/21749064)
* **0** of **12** **(0.0%)** changed or added relevant lines in **1**
shaofengshi merged pull request #472: KYLIN-3819 kylin.sh run in the foreground
URL: https://github.com/apache/kylin/pull/472
This is an automated message from the Apache Git Service.
To respond to the message, please log on
shaofengshi merged pull request #466: KYLIN-3786 Add integration test for
realtime-streaming
URL: https://github.com/apache/kylin/pull/466
This is an automated message from the Apache Git Service.
To respond to the message,
asfgit commented on issue #475: kylin - livy
URL: https://github.com/apache/kylin/pull/475#issuecomment-465926318
Can one of the admins verify this patch?
This is an automated message from the Apache Git Service.
To respond
javalife0312 opened a new pull request #475: kylin - livy
URL: https://github.com/apache/kylin/pull/475
livy submit spark-job
This is an automated message from the Apache Git Service.
To respond to the message, please log on
Sidonet commented on issue #474: KYLIN-3537
URL: https://github.com/apache/kylin/pull/474#issuecomment-465982746
Hello Shaofengshi!
Yes you are right, i've moved PR to correct branch.
Thanks!
This is an automated
shaofengshi commented on issue #474: KYLIN-3537
URL: https://github.com/apache/kylin/pull/474#issuecomment-465973503
Alexander, thanks for the PR! I think this is only for Hadoop 3/HBase 2
right? If so, the change should go to "master-hadoop3.1" branch. Just let me
know so I can move the
shaofengshi commented on issue #473: add Apache find-spark-home fallback script
URL: https://github.com/apache/kylin/pull/473#issuecomment-465974613
Hello Shopee, could you please report a JIRA to describe this enhancement?
Thank you!
Click "Create" and then select "KYLIN" as the
codecov-io edited a comment on issue #474: KYLIN-3537
URL: https://github.com/apache/kylin/pull/474#issuecomment-465703293
# [Codecov](https://codecov.io/gh/apache/kylin/pull/474?src=pr=h1) Report
> :exclamation: No coverage uploaded for pull request base
(`master-hadoop3.1@d090367`).
shopee-jin opened a new pull request #473: add Apache find-spark-home fallback
script
URL: https://github.com/apache/kylin/pull/473
https://github.com/apache/spark/blob/master/bin/find-spark-home
This is an automated message
asfgit commented on issue #473: add Apache find-spark-home fallback script
URL: https://github.com/apache/kylin/pull/473#issuecomment-465622772
Can one of the admins verify this patch?
This is an automated message from the
asfgit commented on issue #474: KYLIN-3537
URL: https://github.com/apache/kylin/pull/474#issuecomment-465661626
Can one of the admins verify this patch?
This is an automated message from the Apache Git Service.
To respond to
Sidonet opened a new pull request #474: KYLIN-3537
URL: https://github.com/apache/kylin/pull/474
Use Spark to build Cube on Yarn faild at Setp8 on HDP3.0
add HBase dependicies to avoid java.lang.NoClassDefFoundError
This
codecov-io commented on issue #473: add Apache find-spark-home fallback script
URL: https://github.com/apache/kylin/pull/473#issuecomment-465665951
# [Codecov](https://codecov.io/gh/apache/kylin/pull/473?src=pr=h1) Report
> :exclamation: No coverage uploaded for pull request base
codecov-io commented on issue #474: KYLIN-3537
URL: https://github.com/apache/kylin/pull/474#issuecomment-465703293
# [Codecov](https://codecov.io/gh/apache/kylin/pull/474?src=pr=h1) Report
> :exclamation: No coverage uploaded for pull request base
(`master@6eee273`). [Click here to
Aaron commented on issue #488: KYLIN-3820 Add a curator-based scheduler
URL: https://github.com/apache/kylin/pull/488#issuecomment-467715454
retest this please
This is an automated message from the Apache Git Service.
To
coveralls edited a comment on issue #484: KYLIN-3820 Add a curator-based
scheduler
URL: https://github.com/apache/kylin/pull/484#issuecomment-467030756
## Pull Request Test Coverage Report for [Build
4168](https://coveralls.io/builds/21871044)
* **114** of **177** **(64.41%)**
1001 - 1100 of 9758 matches
Mail list logo