Alanxtl commented on PR #3357:
URL: https://github.com/apache/dubbo-go/pull/3357#issuecomment-4585678280
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]
