wwbmmm commented on PR #2358:
URL: https://github.com/apache/brpc/pull/2358#issuecomment-1735415459
上面提了一些小问题。
另外想探讨一下整体的设计：
1.
目前看来Server端分tag需要依赖于Client端设置connection_group，在Socket中把tag相关信息传过来。但是这样的话存在问题：
* 修改了通信协议，强依赖于支持该特性的Client，无法兼容以前的Client
* 让Client来决定Server的worker tag是否合理？按理说worker
tag应该由server端的业务逻辑决定。如果server想要调整worker tag，还需要修改所有的Client，这也不方便吧
针对这个问题，我的建议是这样：
* 对于请求处理逻辑中的隔离，可以在用户callback里，自己指定tag调用bthread_start_xxx来将任务发送到指定的worker
* 对于IO的隔离，可以考虑按照端口来隔离，不同端口的Server指定不同的tag。
* 可以把bthread支持多tag和brpc支持多worker池的PR分成两个，我理解前者更没有争议一些
2.
关于bthread支持多tag，是否考虑支持不同的tag指定不同的worker数量呢？毕竟不同tag执行的业务逻辑性质会有较大的差异，对worker数量的需求应该是不一样的
--
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: dev-unsubscr...@brpc.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@brpc.apache.org
For additional commands, e-mail: dev-h...@brpc.apache.org
