misakacoder
misakacoder
@KomachiSion 这个bug我重新提交了一个pr，你看看有没有问题
> 方案中可以认可的点： > > 1. 用SPI进行扩展。 > 2. 将AddressServer和Properties独立作为两个拓展实现。 > 3. 将公共逻辑抽象到抽象类中 > > 可能有问题的点： > > 1. SPI不需要自己实现，我理解jdk自带的足以支持 > 2. order的排序方式可以写入到AbstractServerListManager，作为方法提供，用注解+反射的方法可能没那么好。 > 3. 事件不需要修改名称， 旧的名称足以表达意思 > 4. 地址服务器的实现可能有逻辑遗漏，建议在梳理一下，做好注册和配置中心的区分（命名和调用逻辑上。 1....
@KomachiSion 幸苦空了看看我的回复和疑惑，我这边好调整调整
> @misakacoder any update？ 不好意思，前几天有点忙，我调整了下，幸苦帮忙review下呢
@KomachiSion 大佬，我优化了一下，麻烦再review一下呢
@KomachiSion 已修改，麻烦再review下呢，另外NamingHttpClientProxy只有一个地址就不重试的逻辑我是优化成这样了。 
> 目前初步review似乎没有太大的问题了， 之后会在活动截止前对PR进行整体评判后决定是否合入。 好的，感谢