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

Introduce high-level MemcachedClient #101

Merged
merged 1 commit into from
Jan 15, 2024
Merged

Introduce high-level MemcachedClient #101

merged 1 commit into from
Jan 15, 2024

Conversation

56quarters
Copy link
Owner

Create a high-level client that interacts with multiple Memcached servers depending on the key or operation being performed. The client uses rendezvous hashing to determine which server is responsible for any relevant keys.

In addition the following improvements have been made:

  • Key type introduced to validate keys before using them
  • mc has been rewritten to clean up resources properly

The following things are missing and will done in follow up work:

  • Tests for newly introduced client and hashing
  • Type-safe alternative to string hostnames

Create a high-level client that interacts with multiple Memcached servers
depending on the key or operation being performed. The client uses rendezvous
hashing to determine which server is responsible for any relevant keys.

In addition the following improvements have been made:
* `Key` type introduced to validate keys before using them
* `mc` has been rewritten to clean up resources properly

The following things are missing and will done in follow up work:
* Tests for newly introduced client and hashing
* Type-safe alternative to string hostnames
@56quarters 56quarters marked this pull request as ready for review January 15, 2024 21:15
@56quarters 56quarters merged commit 12f0947 into master Jan 15, 2024
5 checks passed
@56quarters 56quarters deleted the client branch January 15, 2024 21:15
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.

1 participant