You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@dubbo.apache.org by GitBox <gi...@apache.org> on 2022/11/02 02:50:48 UTC

[GitHub] [dubbo] owen200008 commented on pull request #10864: fix_providerby_change_discovery_error

owen200008 commented on PR #10864:
URL: https://github.com/apache/dubbo/pull/10864#issuecomment-1299492928

   > 不太建议通过添加一个新的缓存来解决,因为这只能解一个特定场景的问题,如果有两个 `provided-by` 不一样还是解决不了的。 建议可以把读缓存的逻辑后置,优先直读 `provided-by` 然后再去考虑命中缓存的。
   
   我完全同意  
   我提交第一版本改动是这个方向,没成功,读取缓存地方挺多,昨天审视了下,把控不准缓存使用的场景
   
   我试着改下
   
   我们有有挺多业务是一个接口多个实现的
   所以我肯定是完全同意你上面说的改法


-- 
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: notifications-unsubscribe@dubbo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@dubbo.apache.org
For additional commands, e-mail: notifications-help@dubbo.apache.org