Github user mgaido91 commented on a diff in the pull request:

    https://github.com/apache/spark/pull/20174#discussion_r160772523
  
    --- Diff: 
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/ReplaceOperatorSuite.scala
 ---
    @@ -198,6 +198,20 @@ class ReplaceOperatorSuite extends PlanTest {
         comparePlans(optimized, correctAnswer)
       }
     
    +  test("add one grouping key if necessary when replace Deduplicate with 
Aggregate") {
    +    val input = LocalRelation()
    +    val query = Deduplicate(Seq.empty, input) // dropDuplicates()
    +    val optimized = Optimize.execute(query.analyze)
    +
    +    val correctAnswer =
    --- End diff --
    
    nit: this can be all in one line


---

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

Reply via email to