Skip to content

Conversation

rainerschoe
Copy link
Member

@rainerschoe rainerschoe commented Sep 3, 2025

Refactor the hashing code a little + add test to ensure we use the correct hashing algorithm.

closes #2

Signed-off-by: Travis <[email protected]>
Signed-off-by: Travis <[email protected]>
Signed-off-by: Travis <[email protected]>
Signed-off-by: Travis <[email protected]>
Copy link
Collaborator

@jgsogo jgsogo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 I'm fine closing #2. It's implemented. #16 should be the one that provides coverage.

@rainerschoe rainerschoe merged commit 95d5e21 into main Sep 8, 2025
3 checks passed
@rainerschoe rainerschoe deleted the murmur_refact branch September 8, 2025 08:47
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

Successfully merging this pull request may close these issues.

Use Murmurhash algorithm for the phased rollout
2 participants