wwbmmm commented on a change in pull request #1559:
URL: https://github.com/apache/incubator-brpc/pull/1559#discussion_r750852274
##########
File path: docs/cn/client.md
##########
@@ -299,7 +311,11 @@ if (cntl->Failed()) {
你可以独立地创建这些对象，并使用[NewCallback](#使用NewCallback)生成done，也可以把Response和Controller作为done的成员变量，[一起new出来](#继承google::protobuf::Closure)，一般使用前一种方法。
-**发起异步请求后Request和Channel也可以立刻析构**。这两样和response/controller是不同的。注意:这是说Channel的析构可以立刻发生在CallMethod**之后**，并不是说析构可以和CallMethod同时发生，删除正被另一个线程使用的Channel是未定义行为（很可能crash）。
+发起异步请求后Request可以立刻析构。(SelectiveChannel是个例外，SelectiveChannel情况下必须在请求处理完成后再释放request对象）
+
+发起异步请求后(CallMethod之后），不建议立即析构Channel。（由于brpc当前实现的[一个bug](https://github.com/apache/incubator-brpc/issues/658)，在异步请求结束前析构Channel有小概率crash）
Review comment:
ok, 已修改
--
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]
