xxs588 commented on PR #3269: URL: https://github.com/apache/dubbo-go/pull/3269#issuecomment-4110716644
修复了一下冲突，先 fetch了develop，然后对当前分支做了 rebase。 过程中遇到的冲突是预期内的 modify/delete（filter/hystrix/），因为 develop分支已经把这部分迁移并删除了。 然后我保持 develop分支 的删除结果，移除本分支对应的 hystrix 改动，然后继续变基，只保留本 PR 本身要交的逻辑 另外我排查到一个并发测试点 directory.go 里已经有 snapshotCacheInvokers() 这个并发安全读路径，但是测试里有直接读取 cacheInvokers 的写法 这样子会绕过同步语义并触发 race。 但是这个属于测试侧读取模型问题，不应该在 pr 3269 里面修复所以为了保持 PR 原子性，我切到 split/3247-registry-lifecycle 单独提交一个最小测试补丁，只修复了一下测试读取路径 -- 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]
