Github user hvanhovell commented on the issue:

    https://github.com/apache/spark/pull/20568
  
    @mrkm4ntr I see your point. Adding a method to Murmur3 would work.
    
    The problem is that we are now going to release a `FeatureHasher` in Spark 
2.3 that uses the current Murmur3 implementation. If we change this to use the 
correct Murmur3 implementation after the release of Spark 2.3 we will break all 
models using feature hashing created using Spark 2.3. This might be a blocker. 
Can you send an e-mail to the dev list?
    
    cc @sameeragarwal @srowen for more visibility.


---

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

Reply via email to