[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-12 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-130184621 [Test build #40597 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40597/consoleFull) for PR 8038 at commit

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-12 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-130198626 [Test build #1478 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1478/consoleFull) for PR 8038 at commit

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-12 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-130198509 [Test build #1477 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1477/consoleFull) for PR 8038 at commit

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-130183567 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-130183602 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-12 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-130249673 **[Test build #1477 timed out](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1477/console)** for PR 8038 at commit

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-12 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-130249780 **[Test build #1478 timed out](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1478/console)** for PR 8038 at commit

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-12 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-130232958 **[Test build #40597 timed out](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40597/console)** for PR 8038 at commit

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-130233086 Merged build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-12 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-130368244 [Test build #1495 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1495/consoleFull) for PR 8038 at commit

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-12 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-130360096 So weird... this finished running all python tests successfully too but still timed out somehow? --- If your project is set up for it, you can reply to this email

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/8038 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-12 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-130373677 I'm merging this since it's unlikely a separate issue to cause your test timeout (all the tests did run) --- If your project is set up for it, you can reply to this email

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-12 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-130412913 [Test build #1495 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1495/console) for PR 8038 at commit

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-11 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-130082015 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-11 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-130083914 retest this please --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-11 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-130084399 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-11 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-130084424 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-11 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-130085526 Merged build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-11 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-130086882 [Test build #1454 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1454/consoleFull) for PR 8038 at commit

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-11 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-130081992 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-11 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-130087846 [Test build #1454 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1454/console) for PR 8038 at commit

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-11 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-130088415 Merged build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-11 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-130098465 [Test build #1463 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1463/consoleFull) for PR 8038 at commit

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-11 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-130098613 I just triggered another build as a backup. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-11 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-129964232 [Test build #1452 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1452/consoleFull) for PR 8038 at commit

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-11 Thread andrewor14
Github user andrewor14 commented on a diff in the pull request: https://github.com/apache/spark/pull/8038#discussion_r36769541 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/aggregate/TungstenAggregationIteratorSuite.scala --- @@ -0,0 +1,52 @@ +/* + *

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-11 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-129975022 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-11 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-129974982 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-11 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-129976989 [Test build #40472 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40472/consoleFull) for PR 8038 at commit

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-11 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-130037912 **[Test build #1452 timed out](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1452/console)** for PR 8038 at commit

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-11 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-130050312 **[Test build #40472 timed out](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40472/console)** for PR 8038 at commit

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-11 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-130051505 Merged build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-11 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-130090530 retest this please --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-11 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-130091330 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-11 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-130091313 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-11 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-130091843 [Test build #40525 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40525/consoleFull) for PR 8038 at commit

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-11 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-130109753 compilation failed? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-11 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-130150506 [Test build #1470 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1470/console) for PR 8038 at commit

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-11 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-130153744 ``` [error]

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-11 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-130111442 ``` [error]

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-11 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-130124231 OK, I fixed. Jenkins, retest this please? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-11 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-130137704 [Test build #1470 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1470/consoleFull) for PR 8038 at commit

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-11 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-130125731 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-11 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-130125719 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-11 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-130109271 [Test build #40525 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40525/console) for PR 8038 at commit

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-11 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-130113182 [Test build #1463 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1463/console) for PR 8038 at commit

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-11 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-130136336 Merged build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/8038#discussion_r36682524 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/TungstenAggregate.scala --- @@ -61,32 +62,54 @@ case class TungstenAggregate(

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/8038#discussion_r36682953 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/TungstenAggregate.scala --- @@ -61,32 +62,54 @@ case class TungstenAggregate(

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-129595396 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-129595355 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-129599850 test this please. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/8038#discussion_r36683295 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/TungstenAggregationIterator.scala --- @@ -82,10 +80,12 @@ class

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/8038#discussion_r36684178 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/TungstenAggregationIterator.scala --- @@ -334,7 +334,7 @@ class

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-129615693 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/8038#discussion_r36684365 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/TungstenAggregationIterator.scala --- @@ -426,6 +426,10 @@ class

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/8038#discussion_r36684718 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/TungstenAggregationIterator.scala --- @@ -666,21 +676,20 @@ class

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread andrewor14
Github user andrewor14 commented on a diff in the pull request: https://github.com/apache/spark/pull/8038#discussion_r36686460 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/aggregate/TungstenAggregationIteratorSuite.scala --- @@ -0,0 +1,52 @@ +/* + *

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-129618857 LGTM. Let's wait for jenkins. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-129623661 Merged build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/8038#discussion_r36684847 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/aggregate/TungstenAggregationIteratorSuite.scala --- @@ -0,0 +1,52 @@ +/* + *

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-129616902 [Test build #40328 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40328/consoleFull) for PR 8038 at commit

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-129625157 test this please. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-129627126 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-129627205 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-129627779 [Test build #40334 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40334/consoleFull) for PR 8038 at commit

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread andrewor14
Github user andrewor14 commented on a diff in the pull request: https://github.com/apache/spark/pull/8038#discussion_r36683980 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/TungstenAggregate.scala --- @@ -61,32 +62,54 @@ case class TungstenAggregate(

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread andrewor14
Github user andrewor14 commented on a diff in the pull request: https://github.com/apache/spark/pull/8038#discussion_r36683740 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/TungstenAggregationIterator.scala --- @@ -82,10 +80,12 @@ class

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread andrewor14
Github user andrewor14 commented on a diff in the pull request: https://github.com/apache/spark/pull/8038#discussion_r36683781 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/TungstenAggregate.scala --- @@ -61,32 +62,54 @@ case class TungstenAggregate(

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread andrewor14
Github user andrewor14 commented on a diff in the pull request: https://github.com/apache/spark/pull/8038#discussion_r36683932 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/TungstenAggregate.scala --- @@ -61,32 +62,54 @@ case class TungstenAggregate(

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-129615607 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread andrewor14
Github user andrewor14 commented on a diff in the pull request: https://github.com/apache/spark/pull/8038#discussion_r36687431 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/TungstenAggregationIterator.scala --- @@ -426,6 +426,10 @@ class

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-129623607 [Test build #40328 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40328/console) for PR 8038 at commit

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-129648450 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-129649533 [Test build #40352 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40352/consoleFull) for PR 8038 at commit

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-129647948 [Test build #1428 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1428/consoleFull) for PR 8038 at commit

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-129647924 test this please. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-129648137 [Test build #1426 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1426/consoleFull) for PR 8038 at commit

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-129647620 Merged build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-129647592 **[Test build #40317 timed out](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40317/console)** for PR 8038 at commit

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-129648465 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-129596720 Merged build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-129600521 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-129600607 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-129690435 [Test build #1433 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1433/console) for PR 8038 at commit

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-129691330 test this please. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-129691804 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-129691873 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-129693398 Merged build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread andrewor14
Github user andrewor14 commented on a diff in the pull request: https://github.com/apache/spark/pull/8038#discussion_r36709321 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/aggregate/TungstenAggregationIteratorSuite.scala --- @@ -0,0 +1,52 @@ +/* + *

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-129601522 [Test build #40317 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40317/consoleFull) for PR 8038 at commit

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-129683775 **[Test build #1428 timed out](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1428/console)** for PR 8038 at commit

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-129683957 **[Test build #1426 timed out](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1426/console)** for PR 8038 at commit

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-129684871 **[Test build #40352 timed out](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40352/console)** for PR 8038 at commit

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-129684954 Merged build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-129660412 **[Test build #40334 timed out](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40334/console)** for PR 8038 at commit

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-129660446 Merged build finished. Test FAILed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8038#issuecomment-129660905 [Test build #1433 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1433/consoleFull) for PR 8038 at commit

[GitHub] spark pull request: [SPARK-9747] [SQL] Avoid starving an unsafe op...

2015-08-10 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/8038#discussion_r36705577 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/aggregate/TungstenAggregationIteratorSuite.scala --- @@ -0,0 +1,52 @@ +/* + *

  1   2   >