AsperforMias commented on PR #3357: URL: https://github.com/apache/dubbo-go/pull/3357#issuecomment-4585700971
> 1. remote metadata storage： > > * unregister 后如果还有剩余 exported services：是的，应该重新计算 revision、publish app metadata、更新存活 instance 上的 revision。这个方向符合预期。 > * unregister 后如果 exported services 为空：我建议本 issue 里只做本地 revision 置 `0`，remote metadata 保留历史 revision 数据。因为没有 live instance 再引用旧 revision 了；remote unpublish / cleanup 应该放到 cleanup policy 那个独立问题里，不要在 `#3349` 里扩 scope。 > 2. cleanup 顺序： > > * `UnSubscribe`：是的，先 `serviceNameMapping.Remove(url)`，成功后再删本地 subscribed metadata。失败就保留本地 metadata。 > * provider `UnRegister`：我倾向选 A，也就是按实际成功结果同步 cleanup。成功 unregister 的删 metadata，失败保留的继续留在 metadata。B 的 all-or-nothing 太强，反而会让已经成功下线的 instance 继续残留 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]
