chenzhangyi commented on pull request #1196: URL: https://github.com/apache/incubator-brpc/pull/1196#issuecomment-973784876
> > > > > brpc还在迭代吗，这个ci为什么没有merge > > > > > > > > > > > > 这个实现看起来并没有解决问题， socket->id()并不是全局一致的id。这个不符合一致性hash要解决的问题。 > > > > > > > > > 那能否直接用Socket的地址做hash，地址不可能冲突，重用也是之前的Socket被释放。 > > > > > > SelectiveChannel 并没有 "地址"的概念 > > 我是指用Socket对象的指针，来做hash的key 这里"全局"的描述可能有误导， 全局唯一ID指的是集群内跨进程的唯一ID， 一致性hash的目标是保证在集群内一个key在大部分情况下都访问同一个server。 如果保证不了这点，用一致性hash的意义就不大了。 -- 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]
