thunguo commented on PR #1111: URL: https://github.com/apache/incubator-seata-go/pull/1111#issuecomment-4295625934
> Select(...) 现在虽然保留了旧签名，但在 ConsistentHashLoadBalance 场景下还是会退回到随机：Select(...) -> SelectWithConsistent(..., nil) -> ConsistentHashLoadBalance(nil, ...) -> RandomLoadBalance(...)。这样一来，这个 PR 实际上只修复了 SessionManager 这条内部调用链。 我建议这里把修复补完整一些：保留 SelectWithConsistent(...) 给内部显式传 ring 的场景使用，同时让公开的 Select(...) 在 ConsistentHashLoadBalance 下也能走一个默认的 consistent ring，而不是直接 fallback 到 random。至少也建议补一个测试，锁定 Select(ConsistentHashLoadBalance, ...) 对同一 xid 的稳定选择行为。这样 PR 标题里的 “dispatch consistent_hash” 才算对外也成立。 OK，我改一下，之前只考虑了有被调用的那部分 -- 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]
