jenrryyou opened a new pull request, #3215: URL: https://github.com/apache/brpc/pull/3215
### What problem does this PR solve? Issue Number: resolve Fixes https://github.com/apache/brpc/pull/2754 Problem Summary: 批量创建stream场景下，解决 Stream SetConnected() 与 Consume() 并发导致的 SendFeedback 发送与已消费字节统计的竞态问题，使得反馈的已消费字节数准确，避免发送端错误反压。 ### What is changed and the side effects? Changed: Side effects: - Performance effects: 1. 对非批量创建场景，会走到fast path，对性能无任何影响； 2. 对批量创建场景，连接建立前的消费路径会略增 CPU，但只发生在“建连窗口期”，创建完后对性能无任何影响。 - Breaking backward compatibility: 兼容 --- ### Check List: - Please make sure your changes are compilable. - When providing us with a new feature, it is best to add related tests. - Please follow [Contributor Covenant Code of Conduct](https://github.com/apache/brpc/blob/master/CODE_OF_CONDUCT.md). -- 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]
