Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add murmur3? #10

Open
barkanido opened this issue Sep 16, 2019 · 1 comment
Open

add murmur3? #10

barkanido opened this issue Sep 16, 2019 · 1 comment

Comments

@barkanido
Copy link

No description provided.

@achille-roussel
Copy link
Contributor

Thanks for the suggestion!

There are good murmur3 implementation in Go already, for example https://github.com/spaolacci/murmur3 and https://github.com/twmb/murmur3, so I'm not sure what we'd bring to the table by adding support for murmur3 in this package.

If these alternatives are not good enough for your use case, feel free to provide some explanations and why it would make sense for us to maintain a murmur3 hash function in this package, and I'll be happy to consider it!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants