Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/21752
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/21752#discussion_r204885925
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFrameAggregateSuite.scala ---
@@ -717,4 +717,9 @@ class DataFrameAggregateSuite extends QueryT
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/21752#discussion_r204885339
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/RelationalGroupedDataset.scala ---
@@ -470,8 +470,11 @@ class RelationalGroupedDataset protected[s
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/21752#discussion_r204883667
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/RelationalGroupedDataset.scala ---
@@ -470,8 +470,11 @@ class RelationalGroupedDataset protected[s
Github user c-horn commented on a diff in the pull request:
https://github.com/apache/spark/pull/21752#discussion_r202443782
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFrameAggregateSuite.scala ---
@@ -717,4 +717,9 @@ class DataFrameAggregateSuite extends QueryTest
Github user c-horn commented on a diff in the pull request:
https://github.com/apache/spark/pull/21752#discussion_r202445344
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFrameAggregateSuite.scala ---
@@ -717,4 +717,9 @@ class DataFrameAggregateSuite extends QueryTest
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/21752#discussion_r201916394
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFrameAggregateSuite.scala ---
@@ -717,4 +717,9 @@ class DataFrameAggregateSuite extends QueryTest
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/21752#discussion_r201916183
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFrameAggregateSuite.scala ---
@@ -717,4 +717,9 @@ class DataFrameAggregateSuite extends QueryTest
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/21752#discussion_r201916132
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFrameAggregateSuite.scala ---
@@ -717,4 +717,9 @@ class DataFrameAggregateSuite extends QueryTest
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/21752#discussion_r201915580
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFrameAggregateSuite.scala ---
@@ -717,4 +717,9 @@ class DataFrameAggregateSuite extends QueryTest
GitHub user c-horn opened a pull request:
https://github.com/apache/spark/pull/21752
Spark 24788
## What changes were proposed in this pull request?
https://issues.apache.org/jira/browse/SPARK-24788
Add a special case for unresolved expressions when constructing toS
11 matches
Mail list logo