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.

Sign up for GitHub

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

Jump to bottom

proposal: add a new "http" metadata protocol #29

Open

doujiang24 opened this issue Dec 10, 2024 · 1 comment · May be fixed by #37

Contributor

doujiang24 commented Dec 10, 2024

Glad to see #21 start to support multiple metadata servers.

Now, only etcd and redis are supported. I'm here to proposal a new "http" metadata type, benifits:

http metadata server are easy to deploy and scale.
the http metadata server could expose more APIs for system operator know/manage the system better.

Design:
The http server provide three restful APIs:

GET $metadata_uri?key=$key
PUT $$metadata_uri?key=$key, with json body
DELETE $metadata_uri?key=$key

If this does make sense, I'd like to have a try, thanks.

Collaborator

alogfans commented Dec 10, 2024

Good idea! I think inheriting TransferMetadataImpl in mooncake-transfer-engine/src/transfer_metadata.cpp is enough :)

doujiang24 linked a pull request

that will close this issue

[TransferEngine][feature] add the http metadata server. #37

Open

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