AsperforMias commented on PR #3357:
URL: https://github.com/apache/dubbo-go/pull/3357#issuecomment-4583102606
> cleanup 发生在实际 unregister / remove 成功之前。如果后面的 registry unregister 或
service-app mapping remove 失败，本地 metadata 已经被删了，也会造成状态不一致。
然后这个的话，我也想确认一下：
1. 对于 `UnSubscribe`，我理解应该是：
- 先执行 `serviceNameMapping.Remove(url)`
- 只有成功后才删除本地 subscribed metadata
这个顺序是您要的吗？
2. 对于 provider `UnRegister`，当前 service discovery 这条链路还是 bulk
`UnRegisterService()` 语义，不是 targeted unregister
我现在的处理方向是：
- 先执行实际 unregister
- 只对成功 unregister 的 instances 同步做 metadata cleanup
- 对失败保留的 instances 不改本地 metadata
您这里期望的是哪种？：
- A. 这样按成功结果同步 cleanup 就可以
- B. 需要更强的一致性语义，例如出现部分失败时不更新任何 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]
