Alanxtl commented on PR #3357: URL: https://github.com/apache/dubbo-go/pull/3357#issuecomment-4583039023
1. `UnRegister(url)` 只删当前 URL 的 metadata，但后面仍然调用 `UnRegisterService()`，它会注销 `s.instances` 里的所有实例。这样会出现“metadata 只删一个，registry 注销全部”的状态不一致。 2. `RemoveService` / `RemoveSubscribeURL` 底层是指针相等判断，`url.Clone()` 或逻辑相同但不是同一个指针的 URL 删不掉，旧 URL 仍会残留并参与 revision 计算。 3. #3349 “删除后重新计算并发布应用级 metadata revision”，这个 PR 只刷新了本地 `Revision`，没有处理 remote metadata storage 下的 publish/update。 4. cleanup 发生在实际 unregister / remove 成功之前。如果后面的 registry unregister 或 service-app mapping remove 失败，本地 metadata 已经被删了，也会造成状态不一致。 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
