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

    https://github.com/apache/spark/pull/20521#discussion_r167140801
  
    --- Diff: 
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/HiveExplainSuite.scala
 ---
    @@ -128,32 +128,6 @@ class HiveExplainSuite extends QueryTest with 
SQLTestUtils with TestHiveSingleto
           "src")
       }
     
    -  test("SPARK-17409: The EXPLAIN output of CTAS only shows the analyzed 
plan") {
    --- End diff --
    
    This is kinda a "bad" test. The bug was we optimize the CTAS input query 
twice, but here we are testing the if the EXPLAIN result of CTAS only contains 
analyzed query, which is specific to how we fix that bug at that time.


---

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

Reply via email to