[GitHub] spark issue #21149: [SPARK-24076][SQL] Use different seed in HashAggregate t...

2018-05-08 Thread yucai
Github user yucai commented on the issue: https://github.com/apache/spark/pull/21149 @maropu @hvanhovell thanks very much! --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e

[GitHub] spark issue #21149: [SPARK-24076][SQL] Use different seed in HashAggregate t...

2018-05-08 Thread hvanhovell
Github user hvanhovell commented on the issue: https://github.com/apache/spark/pull/21149 LGTM - merging to master. Thanks! --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands,

[GitHub] spark issue #21149: [SPARK-24076][SQL] Use different seed in HashAggregate t...

2018-05-08 Thread maropu
Github user maropu commented on the issue: https://github.com/apache/spark/pull/21149 ping @hvanhovell @gatorsmile --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail:

[GitHub] spark issue #21149: [SPARK-24076][SQL] Use different seed in HashAggregate t...

2018-04-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21149 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional comma

[GitHub] spark issue #21149: [SPARK-24076][SQL] Use different seed in HashAggregate t...

2018-04-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21149 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89839/ Test PASSed. ---

[GitHub] spark issue #21149: [SPARK-24076][SQL] Use different seed in HashAggregate t...

2018-04-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21149 **[Test build #89839 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89839/testReport)** for PR 21149 at commit [`0818618`](https://github.com/apache/spark/commit/0

[GitHub] spark issue #21149: [SPARK-24076][SQL] Use different seed in HashAggregate t...

2018-04-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21149 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89830/ Test PASSed. ---

[GitHub] spark issue #21149: [SPARK-24076][SQL] Use different seed in HashAggregate t...

2018-04-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21149 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional comma

[GitHub] spark issue #21149: [SPARK-24076][SQL] Use different seed in HashAggregate t...

2018-04-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21149 **[Test build #89830 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89830/testReport)** for PR 21149 at commit [`5e88468`](https://github.com/apache/spark/commit/5

[GitHub] spark issue #21149: [SPARK-24076][SQL] Use different seed in HashAggregate t...

2018-04-25 Thread viirya
Github user viirya commented on the issue: https://github.com/apache/spark/pull/21149 Can you also show the screenshot after this change? --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additio

[GitHub] spark issue #21149: [SPARK-24076][SQL] Use different seed in HashAggregate t...

2018-04-25 Thread maropu
Github user maropu commented on the issue: https://github.com/apache/spark/pull/21149 cc: @hvanhovell --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@

[GitHub] spark issue #21149: [SPARK-24076][SQL] Use different seed in HashAggregate t...

2018-04-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21149 **[Test build #89839 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89839/testReport)** for PR 21149 at commit [`0818618`](https://github.com/apache/spark/commit/08

[GitHub] spark issue #21149: [SPARK-24076][SQL] Use different seed in HashAggregate t...

2018-04-25 Thread yucai
Github user yucai commented on the issue: https://github.com/apache/spark/pull/21149 @maropu Thanks for comments, I have updated, could you help take a look at? --- - To unsubscribe, e-mail: reviews-unsubscr...@spark

[GitHub] spark issue #21149: [SPARK-24076][SQL] Use different seed in HashAggregate t...

2018-04-25 Thread maropu
Github user maropu commented on the issue: https://github.com/apache/spark/pull/21149 oh... good catch. I think you'd be better to put [the detailed info.](https://issues.apache.org/jira/browse/SPARK-24076?focusedCommentId=16451743&page=com.atlassian.jira.plugin.system.issuetabpanels%3

[GitHub] spark issue #21149: [SPARK-24076][SQL] Use different seed in HashAggregate t...

2018-04-25 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21149 **[Test build #89830 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89830/testReport)** for PR 21149 at commit [`5e88468`](https://github.com/apache/spark/commit/5e

[GitHub] spark issue #21149: [SPARK-24076][SQL] Use different seed in HashAggregate t...

2018-04-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21149 Can one of the admins verify this patch? --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional