Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/8015#issuecomment-128837403
The right JIRA number is actually SPARK-9677 by the way!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as wel
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/8015
---
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 enab
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/8015#discussion_r36562182
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/TungstenAggregationIterator.scala
---
@@ -397,14 +397,20 @@ class TungstenAggregatio
Github user yhuai commented on the pull request:
https://github.com/apache/spark/pull/8015#issuecomment-128836244
Merging to master and branch 1.5.
---
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 user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/8015#discussion_r36562064
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/TungstenAggregationIterator.scala
---
@@ -397,14 +397,20 @@ class TungstenAggre
Github user yhuai commented on the pull request:
https://github.com/apache/spark/pull/8015#issuecomment-128835530
LGTM
---
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
Github user yhuai commented on the pull request:
https://github.com/apache/spark/pull/8015#issuecomment-128835256
`TungstenAggregationIterator` part looks good.
---
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 proj
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8015#issuecomment-128818136
Merged build finished. Test PASSed.
---
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8015#issuecomment-128817599
[Test build #40177 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40177/console)
for PR 8015 at commit
[`225eac2`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8015#issuecomment-128772104
[Test build #40177 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40177/consoleFull)
for PR 8015 at commit
[`225eac2`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8015#issuecomment-128771665
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 have
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8015#issuecomment-128771644
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 h
Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/8015#issuecomment-128770994
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8015#issuecomment-128603507
[Test build #40132 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40132/console)
for PR 8015 at commit
[`225eac2`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8015#issuecomment-128603714
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 project
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8015#issuecomment-128580067
Merged build finished. Test PASSed.
---
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8015#issuecomment-128579882
[Test build #40123 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40123/console)
for PR 8015 at commit
[`0be3a42`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8015#issuecomment-128567155
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 have
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8015#issuecomment-128567367
[Test build #40132 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40132/consoleFull)
for PR 8015 at commit
[`225eac2`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8015#issuecomment-128567089
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 h
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8015#issuecomment-128558836
[Test build #40123 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40123/consoleFull)
for PR 8015 at commit
[`0be3a42`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8015#issuecomment-128558199
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8015#issuecomment-128558237
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 have
GitHub user andrewor14 opened a pull request:
https://github.com/apache/spark/pull/8015
[SPARK-9674] Re-enable ignored test in SQLQuerySuite
The original code that this test tests is removed in
https://github.com/apache/spark/commit/9270bd06fd0b16892e3f37213b5bc7813ea11fdd.
It was
24 matches
Mail list logo