Albumen Kevin
Albumen Kevin
@a364176773 > 个人暂时不倾向增加module的方式,我们倾向后期弄个seata-integration仓库,不再将integration放在seata仓库中随seata版本发版,而是拆开更灵活的根据微服务组件如果dubbo.grpc等迭代而迭代,这样用户仅需引入对应的seata-dubbo依赖即可 这个我暂时没看到有对应建好的仓库,所以提交到这里了,如果有更高的存放地方请和我说 > 另外问一嘴,RpcContext.getContext()以及 RpcContext.getServerContext()在dubbo3上已经不需要手动清除了是吗? 是的,但是由于 seata 是基于 filter 来做的扩展点,不建议还使用 RpcContext,否则很容易和 Dubbo 处理 RpcContext 的 filter 出现顺序影响 > 有没有什么别的方式可以知道当前dubbo的version,加个if,让ApacheDubboTransactionPropagationFilter不去调用会出现NoSuchMethodError的这俩方法是不是也可以? 这个 PR 里面不止修改了 RpcContext 的使用,还修改了 Filter 实现的扩展点,客户端侧从 Filter 改成了选址前的...
> 至于说的不建议使用RpcContext,据我所知某些dubbo的版本RpcContext不一定会清除相关的隐式传参的内容,所以我们这边是统一加上,如果说会跟dubbo自身的处理起冲突,是否可以调整order? 这里直接不要使用了不就好了,另外 Dubbo 的 RpcContext 处理优先级是最高的,其他 Filter 很难提高优先级,而且不好说未来 dubbo 这块又需要调整,然后就裂开了,从 invocation 传就没问题了
> > @a364176773 > > > 个人暂时不倾向增加module的方式,我们倾向后期弄个seata-integration仓库,不再将integration放在seata仓库中随seata版本发版,而是拆开更灵活的根据微服务组件如果dubbo.grpc等迭代而迭代,这样用户仅需引入对应的seata-dubbo依赖即可 > > > > > > 这个我暂时没看到有对应建好的仓库,所以提交到这里了,如果有更高的存放地方请和我说 > > > 另外问一嘴,RpcContext.getContext()以及 RpcContext.getServerContext()在dubbo3上已经不需要手动清除了是吗? > > > > > > 是的,但是由于 seata 是基于 filter 来做的扩展点,不建议还使用...
> 我个人想法是这个pr最好能兼容2.7.x和3.x,而不是另起module,如果是的,等我们弄个新的仓库,或者把这块从seata-allmodule里移除掉,将各自对应的版本重新改版本号,比如现在的seata中的alibabadubbo模块直接发个2.5.x的版本后删掉,再将apache-dubbo版本保留,发一个2.7.x的版本,直接迭代到3.x,发一个3.x的版本,未来直接跟dubbo的版本号走,再在官网标注使用不同微服务框架需要使用的xid传递的module的版本号,总的来说2个事 1.拆模块 2.跟apache-dubbo版本走 话说如果拆掉,可以把这个仓库直接放到dubbo下吗,就跟spring-cloud-alibaba那边一样,我们参与一起维护?这样就最好了版本号都是跟着dubbo自身走,用户用的也比较放心 如果放到seata侧要再等一下,等拆出来后,河清兄你这边再交个代码,我们整进来发个3.x的seata-dubbo 版本迁移的我前面回复了,出个兼容 2.7.x 的版本是可以的,但是 Dubbo 3 下面会有性能下降,可以一步一步走 放到 dubbo 仓库也是可以的,可以放到 apache/dubbo-spi-extensions 下面,但是后续 seata 发版本也需要测试和 Dubbo 侧的兼容性,Dubbo 这边合入 dubbo-spi-extensions 后每个小迭代都会测试兼容性
> > Fixed in 3.1.0 > > 想问下 为什么在3.x内比较的时候要忽略id属性, 这是出于什么原因考虑的? 在2.x内最新版也是没有忽略id属性的 因为 id 在大部分是时候是自动生成的,而 equal 比较的时候是比较的属性
因为要降低注册中心的负载,3.x 版本应该默认不会注册了
你用的是什么注册中心
Can you please submit a PR to fix it? It is needed to check if `executorService` is being created by `org/apache/dubbo/metadata/AbstractCacheManager.java:72`.
> 影响了整体的其他api调用 这个 ip 也发布了其他服务嘛
其他服务是什么报错