-
Notifications
You must be signed in to change notification settings - Fork 5.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add brpc support. #10804
Comments
"to split data transformation and rpc logic." => to decouple RPC communication and operators |
Overall, I have no objection for trying out brpc. (still not sure if we should do it now or after 7.5) However, given our hard deadline to have good distributed training by 6.20, we need to be very very careful about task assignment and timing. I suggest we allocate 1 RD for trying out brpc, while others keep optimizing the common distributed training codes and existing grpc. There are still many things to improve before the 6.20 deadline: |
您好,此issue在近一个月内暂无更新,我们将于今天内关闭。若在关闭后您仍需跟进提问,可重新开启此问题,我们将在24小时内回复您。因关闭带来的不便我们深表歉意,请您谅解~感谢您对PaddlePaddle的支持! |
Since you haven't replied for more than a year, we have closed this issue/pr. |
rpc_server
rpc_client
interface to decouple RPC communication and operatorssync_mode_
scope_
from grpc server to operators which should care about them.Omit the argumentctx
in functions such asAsyncSendVariable
?Cleanup deserialization.brpc
supportComptable with grpc::ByteBuffer and brpc::IOBufbrpc
implementation ofrpc_server
rpc_client
.brpc
unit test.The text was updated successfully, but these errors were encountered: