Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/9556#discussion_r44351475
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/FunctionRegistry.scala
---
@@ -177,6 +178,7 @@ object FunctionRegistry {
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/9556#discussion_r44352010
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/expressions/WindowSpec.scala ---
@@ -141,40 +141,56 @@ class WindowSpec private[sql](
*/
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/9556#discussion_r44352031
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/functions.scala ---
@@ -53,6 +54,12 @@ object functions {
private def withExpr(expr:
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/9556#discussion_r44352491
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/SQLConf.scala ---
@@ -448,15 +448,17 @@ private[spark] object SQLConf {
defaultValue =
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/9556#discussion_r44352445
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/GroupedData.scala ---
@@ -88,30 +89,33 @@ class GroupedData protected[sql](
namedExpr
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/9556#issuecomment-155238823
Yay, death to Aggregation1!
---
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 marmbrus commented on the pull request:
https://github.com/apache/spark/pull/9556#issuecomment-155238877
this is going to conflict at least logically with #9499
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/9556#discussion_r44343490
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkStrategies.scala ---
@@ -146,148 +146,105 @@ private[sql] abstract class
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/9556#discussion_r44344541
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkStrategies.scala ---
@@ -146,148 +146,105 @@ private[sql] abstract class SparkStrategies
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9556#issuecomment-155228625
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 user yhuai commented on the pull request:
https://github.com/apache/spark/pull/9556#issuecomment-155231968
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9556#issuecomment-155232902
**[Test build #45442 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45442/consoleFull)**
for PR 9556 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9556#issuecomment-155014247
**[Test build #45350 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45350/consoleFull)**
for PR 9556 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9556#issuecomment-155014293
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 user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/9556#discussion_r44267784
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkStrategies.scala ---
@@ -146,148 +146,105 @@ private[sql] abstract class
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/9556#discussion_r44268074
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkStrategies.scala ---
@@ -146,148 +146,105 @@ private[sql] abstract class
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/9556#discussion_r44268156
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/TungstenAggregate.scala
---
@@ -21,22 +21,22 @@ import
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/9556#discussion_r44268135
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/SortBasedAggregate.scala
---
@@ -27,15 +27,15 @@ import
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9556#issuecomment-154922605
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 user rxin commented on the pull request:
https://github.com/apache/spark/pull/9556#issuecomment-154929171
YAY.
---
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9556#issuecomment-154950298
**[Test build #45342 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45342/consoleFull)**
for PR 9556 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9556#issuecomment-154950509
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9556#issuecomment-154922620
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 user hvanhovell commented on the pull request:
https://github.com/apache/spark/pull/9556#issuecomment-154984531
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 yhuai opened a pull request:
https://github.com/apache/spark/pull/9556
[SPARK-9830] [SQL] Remove AggregateExpression1 and Aggregate Operator used
to evaluate AggregateExpression1s
https://issues.apache.org/jira/browse/SPARK-9830
This PR removes
Github user yhuai commented on the pull request:
https://github.com/apache/spark/pull/9556#issuecomment-154930384
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9556#issuecomment-154930324
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9556#issuecomment-154930683
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9556#issuecomment-154930969
**[Test build #45342 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45342/consoleFull)**
for PR 9556 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9556#issuecomment-154930666
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9556#issuecomment-154989888
**[Test build #45350 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45350/consoleFull)**
for PR 9556 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9556#issuecomment-154986197
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
101 - 132 of 132 matches
Mail list logo
