zhengJade commented on PR #2819: URL: https://github.com/apache/brpc/pull/2819#issuecomment-2716289023
> > PR #2907 单独实现了第二个优化，看起来更优雅一些。 第一个优化可以再讨论一下 @chenBright > > @zhengJade #2907 已经合了，可以rebase master更新一下这个PR吗？ > > 可以再测一下，观察#2897 加的指标，看看epoll调度是否变快了吗？ @chenBright 单独的第一个优化 #2907 其实没有任何优化，对系统的调用其实也没有减少，因为 brpc 的 worker 在 waiting 和 runnning 的状态之间切换十分频繁，所以 waiting count 总是大于 0，在我多次的测试下也发现，几百次的 wake 也就能触发一次 优化，所以我之前就不认为单独合入第一个优化有意义，感觉是为了合入而合入。我看看 #2897 咋用。 -- 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
