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

Add redis backed cache implementation as alternative to infinispan #3532

Open
StFS opened this issue Aug 23, 2023 · 3 comments
Open

Add redis backed cache implementation as alternative to infinispan #3532

StFS opened this issue Aug 23, 2023 · 3 comments
Assignees
Labels
C&C Command and Control enhancement

Comments

@StFS
Copy link
Contributor

StFS commented Aug 23, 2023

I think providing a Redis alternative for distributed caching solution in Hono in addition to the current Infinispan implementation would be beneficial in many ways.

First, many companies already have a Redis cluster available as opposed to Infinispan which is not nearly as widely used. Second, all the major cloud providers offer some sort of a managed Redis solution (ElastiCache, Azure Cache for Redis, Memorystore) which makes it much easier to set up a Redis cluster than setting up an Infinispan grid.

This ticket is mainly created for communication purposes and to serve as a hub for information gathering.

I may take a stab at creating a PR for this.

@StFS StFS changed the title Add redis backed cache as alternative to using infinispan Add redis backed cache implementation as alternative to infinispan Aug 23, 2023
@sophokles73 sophokles73 added enhancement C&C Command and Control labels Sep 21, 2023
@StFS
Copy link
Contributor Author

StFS commented Apr 10, 2024

Current status of this issue is that the implementation is underway but has been split into two PRs:

@sophokles73
Copy link
Contributor

BTW are you aware of the struggles around Redis forks: https://lwn.net/Articles/966631/

@StFS
Copy link
Contributor Author

StFS commented Apr 12, 2024

BTW are you aware of the struggles around Redis forks: https://lwn.net/Articles/966631/

Ugh... I was not.

Sounds like this is heading in a similar direction as MongoDB. At any rate, I don't really think it has much impact on my work on this. For now, different solutions haven't diverged much so I assume my implementation would be a good basis for chasing any Redis alternative (be it Valkey or something else). Probably, the deciding factor as is would be if the quarkus-redis-client extension got modified in such a way to not be compatible with some of these alternatives.

My objective with this is pretty clear: to support whatever variation of Redis that AWS ElastiCache uses. I think that would bring most value for Hono users as it's the largest cloud provider. My hope would be that Google and Microsoft would use the same variant as AWS but I guess that remains to be seen.

Would you say that this news should impact my work on this in any way? I would assume that license wise, it shouldn't make much of a difference since MongoDB is used in Hono and that has gone through similar changes.

sophokles73 pushed a commit that referenced this issue May 17, 2024
* Split client-device-connection-infinispan module into client-device-connection-base and client-device-connection-infinispan.
* Moved common classes from client-device-connection-infinispan module into client-device-connection-base module. 
* Split services/command-router module into services/command-router-base and services/command-router-infinispan.
* Updated push_hono_images.sh  script to consider new image names
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C&C Command and Control enhancement
Projects
None yet
Development

No branches or pull requests

2 participants