brpc icon indicating copy to clipboard operation
brpc copied to clipboard

h2协议支持 rdma :直接改SupportedByRdma 是否可以

Open crazypaul123 opened this issue 7 months ago • 3 comments

您好,当前由于工作需要,想要使用h2协议来支持rdma,我这样直接改是不是就可以,测试未发现什么问题

bool SupportedByRdma(std::string protocol) { if ((protocol.compare("baidu_std") == 0) || (protocol.compare("h2") == 0)) { // Since rdma is used for high performance scenario, // we consider baidu_std for the only protocol to support. return true; } return false; }

crazypaul123 avatar May 12 '25 12:05 crazypaul123

Cc @Tuvie

Huixxi avatar May 18 '25 01:05 Huixxi

我觉得是可以的,rdma的逻辑里面没看到有专门baidu_std相关的逻辑。你可以按照上面的修改一下,测试一下。

yanglimingcn avatar May 22 '25 09:05 yanglimingcn

我觉得是可以的,rdma的逻辑里面没看到有专门baidu_std相关的逻辑。你可以按照上面的修改一下,测试一下。

我自己测试了,好像没发现什么问题

crazypaul123 avatar May 26 '25 09:05 crazypaul123