apache/dubbo

ZookeeperRegistryNotifier 无法实现多次通知合并的功能？

开放

#11,257 创建于 2023年1月9日

在 GitHub 查看贡献者指南
 (8 条评论) (0 个反应) (1 位负责人)Java (26,453 个派生)batch import
help wantedtype/bug

仓库指标

星标
 (41,524 个星标)
PR 合并指标
 (PR 指标待抓取)

描述

不太理解 ZookeeperRegistryNotifier 的 notify() 方法里面，为什么要进行 sleep ？

我一开始以为是这里说的【延迟通知】https://cn.dubbo.apache.org/zh/blog/2022/06/23/%e6%b5%85%e6%9e%90-dubbo-3.0-%e4%b8%ad%e6%8e%a5%e5%8f%a3%e7%ba%a7%e5%9c%b0%e5%9d%80%e6%8e%a8%e9%80%81%e6%80%a7%e8%83%bd%e7%9a%84%e4%bc%98%e5%8c%96/#%E5%BB%B6%E8%BF%9F%E9%80%9A%E7%9F%A5

但是阅读代码之后，发现应该起不到这里说的【延迟通知】的效果。

看了 org.apache.dubbo.registry.RegistryNotifier#notify 的实现，org.apache.dubbo.registry.zookeeper.ZookeeperRegistry.ZookeeperRegistryNotifier#notify 的实现的确有问题，ZookeeperRegistryNotifier实现不了【阻塞期间，多次通知的合并】

贡献者指南

研究方向
比较 ZookeeperRegistryNotifier 中的 notify 方法与基类 RegistryNotifier 的实现，以理解预期的合并行为并找出 bug。
技术栈
java
领域
backendapi
议题类型
缺陷
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
3
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
1-3 小时
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
活跃
清晰度议题是否清楚说明期望改动、验收标准和下一步。
基本清晰
前置要求
JavaApache Dubbo
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
50