Skip to content
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

Rdma typo #1398

Closed
wants to merge 5 commits into from
Closed

Rdma typo #1398

wants to merge 5 commits into from

Conversation

tbago
Copy link
Contributor

@tbago tbago commented May 10, 2021

Fixed some rdam branch typo issues.

主要修改了一些拼写相关问题,
1、自定义头文件需要使用引号包含;
2、制表符替换为空格统一格式;
3、删除了一些无用的变量;
4、增加了rdma编译说明,以前的编译说明在example的rdma_performance目录下,对新手不太友好;
5、修改一些变量的类型,将int赋值给uint8_t不太友好;

@wwbmmm
Copy link
Contributor

wwbmmm commented Aug 9, 2022

Hi, 我们后续计划不再维护单独的RDMA分支,而是将RDMA功能合入master,相关PR:#1836
所以这个PR先close了。如果有关于RDMA的改进建议,可以在新的PR下讨论

@wwbmmm wwbmmm closed this Aug 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants