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

feat: 支持通过SSH代理连接Redis缓存 #787

Open
wants to merge 3 commits into
base: v2
Choose a base branch
from

Conversation

voidint
Copy link

@voidint voidint commented Jul 30, 2024

No description provided.

@houseme
Copy link
Collaborator

houseme commented Jul 30, 2024

单独使用redis实现 cache 接口就好了,后续 redis 准备移除,建议用户自己实现。 @voidint

@voidint
Copy link
Author

voidint commented Jul 30, 2024

单独使用redis实现 cache 接口就好了,后续 redis 准备移除,建议用户自己实现。 @voidint

Cache这个接口是有了,但库使用方也需要一个开箱即用的东西,不能说因为水泥、砂石、砖头、钢筋都有了,就让你我这样的人都去自己造房子。况且,移除 redis 实现是一个破坏性变更。

@houseme
Copy link
Collaborator

houseme commented Jul 30, 2024

单独使用redis实现 cache 接口就好了,后续 redis 准备移除,建议用户自己实现。 @voidint

Cache这个接口是有了,但库使用方也需要一个开箱即用的东西,不能说因为水泥、砂石、砖头、钢筋都有了,就让你我这样的人都去自己造房子。况且,移除 redis 实现是一个破坏性变更。

现在 redis 换包名了

@voidint
Copy link
Author

voidint commented Jul 31, 2024

  • 你指的应该是 import path 从github.com/go-redis/redis/v8改成了github.com/redis/go-redis/v9是吧?这个和移除 redis 实现没必然联系吧?假设现在用的就是最新的github.com/redis/go-redis/v9,那过段时间上游变成了github.com/redis/go-redis/v10,那和当前的现状不是一样吗?

  • 假设把Redis的实现删除了,那 memcache 实现如何处理呢?上游的 memcache 库依然会有升级的可能。

  • 项目依赖库版本不是最新的这种情况太正常不过了,调用方最多也就是多了一个间接依赖。如果因为这样的原因,把 redis 和 memcache 这类开箱即用的实现删除,交给调用方自己去实现,那么结局只有两个:1、大家都自己写 redis 实现,无法做到开箱即用,增加了使用成本;2、直接使用 memory 这个内置实现,在多个实例的情况下,这个实现是没法用的,使用方在不了解的情况下可能还会抱怨这是 bug。

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.

2 participants