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

    https://github.com/apache/spark/pull/20525#discussion_r166540285
  
    --- Diff: 
sql/core/src/test/scala/org/apache/spark/sql/test/DataFrameReaderWriterSuite.scala
 ---
    @@ -301,7 +301,6 @@ class DataFrameReaderWriterSuite extends QueryTest with 
SharedSQLContext with Be
           intercept[AnalysisException] {
             
spark.range(10).write.format("csv").mode("overwrite").partitionBy("id").save(path)
           }
    -      
spark.emptyDataFrame.write.format("parquet").mode("overwrite").save(path)
         }
    --- End diff --
    
    Its not legal to write an empty struct in parquet.  Its explained by Herman 
in [SPARK-20593](https://issues.apache.org/jira/browse/SPARK-20593). 
Previously, we didn't setup a write 
    task for this where as now with this fix we do. 


---

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

Reply via email to