alibaba / jetcache

JetCache is a Java cache framework.
Apache License 2.0
4.96k stars 1.03k forks source link

feat: update spring, support jedis cluster pipeline #869

Closed Roiocam closed 3 months ago

Roiocam commented 4 months ago

Motivation

Resolve #865 at 2.8.x

areyouok commented 3 months ago

现在另一个pr里面已经能比较好put all的pipeline问题,那么升级依赖是否还有必要性?并不是spring、jedis有升级这边就同步升级的,而是仅在必要的时候升级。否则升级2.8了,但是又看不到新功能,反而兼容性会有所变更,意义就不大。

另外,其它的几个修改点,是否互相相关?如果不是,我建议拆一下,否则这个pr太大了,不好review。

Roiocam commented 3 months ago

现在另一个pr里面已经能比较好put all的pipeline问题,那么升级依赖是否还有必要性?

不是非常有必要,主要是 jedis 的 api 可能流畅些,另外升级的时候遇到 redisson 的升级时可能的一些问题,算是提前踩坑了

另外,其它的几个修改点,是否互相相关?如果不是,我建议拆一下,否则这个pr太大了,不好review。

Spring 升级的时候有一些相关的:

除此之外就是有个单测表现得很 flaky,导致 PR 的 CI 过不去。

另一个 PR 可以满足,如果近期没升级 Spring 及相关依赖的计划,那么这个 PR 我就关了

areyouok commented 3 months ago

如果出现重大兼容性问题,比如某个依赖升级版本以后,jetcache无法运行基本功能,会考虑升级