search

ndelvalle / rustdis

Partial Redis server implementation using Rust for learning purposes.
https://docs.rs/rustdis
0 stars 0 forks source link

POC Milestone #16

Closed ndelvalle closed 5 months ago

ndelvalle commented 6 months ago

Commands

gillchristian commented 6 months ago

I think the client sends a few other commands, we could include those as well (MEM was it?) to have an e2e case working. Wdyt?

ndelvalle commented 6 months ago

I think the client sends a few other commands, we could include those as well (MEM was it?) to have an e2e case working. Wdyt?

Definitely. I just wanted to have a place where we could set a "roadmap". I think a good idea is to support a basic CRUD operation from a simple GUI client. So we need to document somewhere what are all the commands require for that so we can start working on them.

ndelvalle commented 5 months ago

These are all the String related methods https://redis.io/commands/?group=string, I think a good new milestone would be to implement all of these. What do you think? We can create a new milestone for it.

Also, readying the redis docs (https://redis.io/docs/data-types/strings/) they mention some basic String commands. I think we should give priority to these ones. WDTY?

I would also think we can close this issue after we merge the last PR that implements KEYS and EXISTS.

gillchristian commented 5 months ago

I think we should also finish the ones that are defined now but have a dummy implementation (returning OK). Then we can close this milestone.

But yeah, I agree, let's do the string ones. Firs the basic ones and then all the rest.