Skip to content
This repository was archived by the owner on May 26, 2026. It is now read-only.

doc: add support for SHA3 hashing algorithm - #41

Open
kailash360 wants to merge 1 commit into
milvus-io:mainfrom
kailash360:feature/sha3
Open

doc: add support for SHA3 hashing algorithm#41
kailash360 wants to merge 1 commit into
milvus-io:mainfrom
kailash360:feature/sha3

Conversation

@kailash360

Copy link
Copy Markdown

Summary

Adds a Milvus Enhancement Proposal (MEP) documenting SHA-3 (SHA3-256) support as a configurable alternative to SHA-256 for internal hashing operations in Milvus.

Related
Implementation PR: milvus-io/milvus#48913
Issue: milvus-io/milvus#25970

What this doc covers

  • Motivation for making the hash algorithm configurable for compliance-sensitive deployments
  • Design of the ComputeHash abstraction in internal/util/crypto
  • New config parameter common.security.hashAlgorithm (default: sha256, supported: sha3)
  • Compatibility analysis — existing deployments unaffected, SHA-256 remains the default
  • Test plan covering unit, paramtable, and telemetry-level tests

Signed-off-by: kailash360 <kailashkejriwal21@gmail.com>
@sre-ci-robot
sre-ci-robot requested review from sunby and zwd1208 April 14, 2026 19:00
@sre-ci-robot

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: kailash360
To complete the pull request process, please assign yanliang567 after the PR has been reviewed.
You can assign the PR to them by writing /assign @yanliang567 in a comment when ready.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@sre-ci-robot

Copy link
Copy Markdown
Contributor

Welcome @kailash360! It looks like this is your first PR to milvus-io/milvus-design-docs 🎉

@kailash360

Copy link
Copy Markdown
Author

@zwd1208 @sunby
Kindly review the design doc and let me know if any changes are needed.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants