You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@mahout.apache.org by "Ted Dunning (JIRA)" <ji...@apache.org> on 2010/09/16 21:39:33 UTC

[jira] Issue Comment Edited: (MAHOUT-503) Bad murmur hash implementation ?!?

    [ https://issues.apache.org/jira/browse/MAHOUT-503?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12910279#action_12910279 ] 

Ted Dunning edited comment on MAHOUT-503 at 9/16/10 3:38 PM:
-------------------------------------------------------------

Note this comment in the test cases for MurmurHash:
{code}
    // test data generated by running MurmurHash2_64.cpp
{code}

I did quite a bit of changing around of Andrej's version of the code in this port and generated test vectors much the way that the article you cite did.

      was (Author: tdunning):
    Note this comment in the test cases for MurmurHash:
{code}
    // test data generated by running MurmurHash2_64.cpp
{code}

  
> Bad murmur hash implementation ?!?
> ----------------------------------
>
>                 Key: MAHOUT-503
>                 URL: https://issues.apache.org/jira/browse/MAHOUT-503
>             Project: Mahout
>          Issue Type: Improvement
>          Components: Classification
>            Reporter: Thomas Sauzedde
>
> It looks like the murmur hash implementation is coming from the original C to Java port (see http://www.getopt.org/murmur/MurmurHash.java)
> According to http://dmy999.com/article/50/murmurhash-2-java-port, (not verified myself), this port doesn't produce the same results than the original C code.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.