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 with
SharedSQLContext {
Row(1, 2, 1) :: Row(2, 2, 2) :: Row(3, 2, 3) :: Nil)
}
+ test("SPARK-24788: RelationalGroupedDataset.toString " +
+ "with unresolved grouping expressions should not throw an exception") {
+ noException should be thrownBy testData.groupBy('key).toString
--- End diff --
How about `assert(testData.groupBy('key).toString === "xxx")` instead of
using Matchers?
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]