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 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 --
    
    this blank space was already here so I assumed it was conventional, I will 
change it.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to