Bright Chen
Bright Chen
> 做成可选项，使用者自己决定 嗯，肯定是做成可选项的。
> thread销毁后也会跟着销毁，不应该无限增长 自己起的线程中使用bthread_keytable吗？
@MJY-HUST 10月节后会发新版本，能赶上新版本修复这个问题吗？
Closed this as completed in #2768.
这样也会影响服务端异步rpc响应吧。Socket::StartWrite里直接调用KeepWrite，会导致Write阻塞，不太合理吧。 加个参数，支持使用bthread_start_urgent原地执行KeepWrite，应该能满足需求。
单纯从Socket::Write接口看就是有可能会阻塞吧。
用bthread_start_urgent效果差不多，但是影响更小？
@xybanpeng 可以使用 #2591 测试一下优化效果
> 我们是否可以采用将done传入progressive reader的方式来实现异步接口，然用户在合适的时候适时调用done->Run()？如果可行那么目前的设计可以cover这种方式。 @TangSiyang2001 异步接口在小包场景下，有use after free的问题。开了asan测试HttpTest.server_end_read_failed有heap-use-after-free报错。以下是asan报告的调用栈： ``` #0 0x7f63659a2b83 in brpc::HttpMessage::SetBodyReader(brpc::ProgressiveReader*) /home/runner/work/brpc/brpc/src/brpc/details/http_message.cpp:384:13 #1 0x6aa3e5 in brpc::policy::HttpContext::ReadProgressiveAttachmentBy(brpc::ProgressiveReader*) /home/runner/work/brpc/brpc/test/../src/brpc/policy/http_rpc_protocol.h:118:16 #2 0x7f636586f505 in brpc::Controller::ReadProgressiveAttachmentBy(brpc::ProgressiveReader*) /home/runner/work/brpc/brpc/src/brpc/controller.cpp:1490:18 #3 0x66a23c in (anonymous namespace)::UploadServiceImpl::UploadFailed(google::protobuf::RpcController*, test::HttpRequest const*,...
可以参考 #2307