valkey-io / valkeymodule-rs

Rust valkey SDK for modules

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Should we use Rust aliases to for compatibility bwn valkey-module and redis-module

dmitrypol opened this issue · comments

As part of creating new valkey-module crate we forked redis-module and renamed a bunch of things. For example RedisString became ValkeyString. Should we implement Rust aliases https://doc.rust-lang.org/reference/items/type-aliases.html to hep with code compatibility?

If someone built a module in Rust for Redis using redis-module they can update their Cargo.toml to reference valkey-module. But they would have to make numerous updates in code. Aliases will simplify that.

I think having those type aliases would be great. If you want, I can come up with a PR