brpc icon indicating copy to clipboard operation
brpc copied to clipboard
verified publisher icon apache

enable brpc use rdma

Open Tuvie opened this issue 3 years ago • 10 comments

Trying to merge rdma implementation into master. This implementation is little different from previous rdma branch. I remove some proved useless code and try to improve the handshake process.

Tuvie avatar Jul 13 '22 11:07 Tuvie

@Tuvie 可以简单介绍一下代码结构和改动思路吗？文件有点多，有个整体的overview的话review起来会方便一些，谢谢。

zyearn avatar Aug 03 '22 08:08 zyearn

后续是否支持同一个进程中启动多个brpc server，并分别绑定到不同的rdma网卡？

tsyw1987 avatar Aug 09 '22 05:08 tsyw1987

后续是否支持同一个进程中启动多个brpc server，并分别绑定到不同的rdma网卡？

这个可以在后续支持

Tuvie avatar Aug 11 '22 11:08 Tuvie

@Tuvie 可以简单介绍一下代码结构和改动思路吗？文件有点多，有个整体的overview的话review起来会方便一些，谢谢。

已在docs中添加相应描述

Tuvie avatar Aug 16 '22 05:08 Tuvie

LGTM

wwbmmm avatar Aug 16 '22 07:08 wwbmmm

@Tuvie 可以简单介绍一下代码结构和改动思路吗？文件有点多，有个整体的overview的话review起来会方便一些，谢谢。

已在docs中添加相应描述

感谢。目前正在review，文件有点多，看完需要些时间。

zyearn avatar Aug 17 '22 19:08 zyearn

LGTM.

@Tuvie 从上次提交到现在有什么bugfix吗？没有的话等 @wwbmmm 看看线上运行的稳定程度，没什么问题的话可以考虑合入了。

zyearn avatar Aug 31 '22 18:08 zyearn

@Tuvie 有conflicts，需要merge master重新提交一下

zyearn avatar Aug 31 '22 18:08 zyearn

LGTM.

@Tuvie 从上次提交到现在有什么bugfix吗？没有的话等 @wwbmmm 看看线上运行的稳定程度，没什么问题的话可以考虑合入了。

目前暂时没有，也可以先等等

Tuvie avatar Sep 03 '22 02:09 Tuvie

@Tuvie 有conflicts，需要merge master重新提交一下

done

Tuvie avatar Sep 05 '22 01:09 Tuvie