Github user hvanhovell commented on the issue:

    https://github.com/apache/spark/pull/20568
  
    @mrkm4ntr The change itself looks pretty reasonable. However I am very 
hesitant to merge this because this will probably break bucketing (it uses 
murmur3 to create the buckets); for example a bucketed table written by Spark 
2.2 cannot be safely read by Spark after the change.
    
    Can you explain what problem you are trying to fix here?


---

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

Reply via email to