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

🚀 [Feature]: Implement Valkey Storage #1361

Closed
3 tasks done
H0llyW00dzZ opened this issue May 4, 2024 · 14 comments · Fixed by #1538
Closed
3 tasks done

🚀 [Feature]: Implement Valkey Storage #1361

H0llyW00dzZ opened this issue May 4, 2024 · 14 comments · Fixed by #1538

Comments

@H0llyW00dzZ
Copy link

Feature Description

This is similar to the Redis link here

Additional Context (optional)

No response

Code Snippet (optional)

No response

Checklist:

  • I agree to follow Fiber's Code of Conduct.
  • I have checked for existing issues that describe my suggestion prior to opening this one.
  • I understand that improperly formatted feature requests may be closed without explanation.
@gaby
Copy link
Member

gaby commented May 4, 2024

@H0llyW00dzZ The Redis and Rueidis clients work with Valkey. Therr's no need for anlther driver.

The valkey-go is a fork of rueidis which we have a storage driver for.

@H0llyW00dzZ
Copy link
Author

@H0llyW00dzZ The Redis and Rueidis clients work with Valkey. Therr's no need for anlther driver.

The valkey-go is a fork of rueidis which we have a storage driver for.

However, that might not work anymore in the future.

@grivera64
Copy link
Member

I think we are at a good point to revisit this issue. According to a December 2024 Redis Blog Post:

And though [Redis and Valkey] were both once based on the same code, they’ve begun to diverge in feature sets,
functionality, capabilities, contributors, and more. Over time, the divergence is only likely to increase.

In this case, it may make sense to officially support Valkey through a separate storage driver. If we do this, we don't need to worry as much about breaking changes between Redis/Rueidis clients and Valkey clients for users that rely on GoFiber's storage module.

Currently, the Valkey client appears to be just enough of the same as Ruedis, so it could be as simple as a copy-and-paste of the existing driver and rename to use Valkey's syntax. This way, once the clients start to vastly diverge, it won't inadvertently break GoFiber applications that upgrade to a future version.

What do you all think of this?

@gaby
Copy link
Member

gaby commented Dec 29, 2024

@grivera64 Agree, the code from the rueidis driver can be copy/pasted, and swap the dependency for https://github.com/valkey-io/valkey-go

@H0llyW00dzZ
Copy link
Author

nice, finally

@gaby
Copy link
Member

gaby commented Jan 7, 2025

@ReneWerner87 We need a release for Valkey

@H0llyW00dzZ
Copy link
Author

@ReneWerner87 We need a release for Valkey

Furthermore, it can improve performance related to Valkey, such as reducing latency caused by regional factors. In contrast to Redis Cloud, where database deployment for caching or as an in-memory database is restricted to specific regions, Valkey can be self-hosted or directly accessed from cloud providers like Google Cloud.

@gaby
Copy link
Member

gaby commented Jan 7, 2025

@H0llyW00dzZ You can self-host Redis too.

I run multiple self-hosted Redis and Redis Cluster instances.

@ReneWerner87
Copy link
Member

@ReneWerner87 We need a release for Valkey

is a 0.0.1 version okay, as there will definitely be a breaking change in the next month due to our adaptation to fiber v3 ?

@H0llyW00dzZ
Copy link
Author

@H0llyW00dzZ You can self-host Redis too.

I run multiple self-hosted Redis and Redis Cluster instances.

@gaby Do you self-host Redis with TLS or not? Beause Self-hosting Redis has limitations compared to using Redis Cloud.

@gaby
Copy link
Member

gaby commented Jan 7, 2025

@ReneWerner87 We need a release for Valkey

is a 0.0.1 version okay, as there will definitely be a breaking change in the next month due to our adaptation to fiber v3 ?

Yes

@ReneWerner87
Copy link
Member

https://github.com/gofiber/storage/releases/tag/valkey%2Fv0.1.0

@gaby
Copy link
Member

gaby commented Jan 7, 2025

@H0llyW00dzZ You can self-host Redis too.
I run multiple self-hosted Redis and Redis Cluster instances.

@gaby Do you self-host Redis with TLS or not? Beause Self-hosting Redis has limitations compared to using Redis Cloud.

With and without TLS. It's the same thing. If you want all the features you have to use Redis Stack. See here https://github.com/redis-stack/redis-stack

@H0llyW00dzZ
Copy link
Author

@ReneWerner87 We need a release for Valkey

is a 0.0.1 version okay, as there will definitely be a breaking change in the next month due to our adaptation to fiber v3 ?

@ReneWerner87 It will be better for both v2 and v3 because Fiber v2 is stable. I've been using it on Kubernetes.

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

Successfully merging a pull request may close this issue.

4 participants