Skip to content

feat(go): remove murmur hash go deps#3217

Merged
chaokunyang merged 1 commit intoapache:mainfrom
chaokunyang:remove_murmur_hash_go_deps
Jan 27, 2026
Merged

feat(go): remove murmur hash go deps#3217
chaokunyang merged 1 commit intoapache:mainfrom
chaokunyang:remove_murmur_hash_go_deps

Conversation

@chaokunyang
Copy link
Collaborator

@chaokunyang chaokunyang commented Jan 27, 2026

Why?

  • Remove the external MurmurHash3 dependency from Go while keeping deterministic hashing behavior used by xlang tests and metadata.

What does this PR do?

  • Add a local MurmurHash3 x64 128-bit implementation with seeded 64/128-bit helpers.
  • Wire Go meta string hashing, struct fingerprints, type def header hashing, and xlang tests to the local functions.
  • Drop the murmur3 module from go/fory/go.mod and go/fory/go.sum.

Related issues

Does this PR introduce any user-facing change?

No.

  • Does this PR introduce any public API change?
  • Does this PR introduce any binary protocol compatibility change?

Benchmark

@chaokunyang chaokunyang force-pushed the remove_murmur_hash_go_deps branch from 3d7d8f1 to e673cb4 Compare January 27, 2026 04:05
@chaokunyang chaokunyang merged commit 642305f into apache:main Jan 27, 2026
56 checks passed
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.

2 participants