Github user cloud-fan commented on a diff in the pull request:

    https://github.com/apache/spark/pull/21291#discussion_r188868678
  
    --- Diff: 
sql/core/src/test/scala/org/apache/spark/sql/execution/debug/DebuggingSuite.scala
 ---
    @@ -34,14 +34,13 @@ class DebuggingSuite extends SparkFunSuite with 
SharedSQLContext {
     
       test("debugCodegen") {
         val res = 
codegenString(spark.range(10).groupBy("id").count().queryExecution.executedPlan)
    -    assert(res.contains("Subtree 1 / 2"))
    -    assert(res.contains("Subtree 2 / 2"))
    +    assert(res.contains("Subtree 1 / 1"))
         assert(res.contains("Object[]"))
       }
     
       test("debugCodegenStringSeq") {
         val res = 
codegenStringSeq(spark.range(10).groupBy("id").count().queryExecution.executedPlan)
    -    assert(res.length == 2)
    +    assert(res.length == 1)
    --- End diff --
    
    ditto


---

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

Reply via email to