Github user DaimonPl commented on the issue:

    https://github.com/apache/spark/pull/16578
  
    @mallman how about adding comment explaining why such workaround was done + 
bug number in parquet-mr ? So in future once that bug is fixed, code can be 
cleaned.
    
    Also maybe it's time to remove "DO NOT MERGE" from title? As I understand 
most of comments were addressed :)
    
    Thank you very much for work on this feature. I must admit that we are 
looking forward to have this merged. For us this will be most important 
improvement in Spark 2.3.0 (I hope it will be part of 2.3.0 :) )


---

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

Reply via email to