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.

Sign up for GitHub

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

Jump to bottom

feat：RPC自定义协议的头部字段长度优化 #19

Closed
578223592 opened this issue Jan 19, 2024 · 7 comments
Closed

feat：RPC自定义协议的头部字段长度优化 #19

578223592 opened this issue Jan 19, 2024 · 7 comments
Assignees
@ildnyy
Labels
enhancement New feature or request good first issue Good for newcomers working 进行中

Comments

@578223592
Copy link
Collaborator

头部的_消息体长度_字段目前是固定4个字节，这个可能设计的不是很合理，考虑使用更灵活的方式。
可以参考protobuf的变长编码、Redis中的对于长度的变长编码，即用标志位来确定_消息体长度_,这样来节约RPC的长度
@578223592 578223592 mentioned this issue Jan 19, 2024
Open
11 tasks
@578223592 578223592 added enhancement New feature or request good first issue Good for newcomers labels Jan 19, 2024
@ildnyy
Copy link
Contributor

ildnyy commented Feb 2, 2024

我想试一下解决这个issue

@578223592
Copy link
Collaborator Author

我想试一下解决这个issue

非常欢迎。或许你可以告诉我你完成的日程安排，如果有任何相关问题可以继续在这里留言，非常乐意解答。

@ildnyy
Copy link
Contributor

ildnyy commented Feb 2, 2024

我想试一下解决这个issue

非常欢迎。或许你可以告诉我你完成的日程安排，如果有任何相关问题可以继续在这里留言，非常乐意解答。

请问项目中使用的muduo boost protobuf分别是什么版本的？我预计在二月底之前完成吧

@578223592
Copy link
Collaborator Author

578223592 commented Feb 5, 2024 •
edited
Loading

我想试一下解决这个issue

非常欢迎。或许你可以告诉我你完成的日程安排，如果有任何相关问题可以继续在这里留言，非常乐意解答。

请问项目中使用的muduo boost protobuf分别是什么版本的？我预计在二月底之前完成吧

我的版本信息在：#8 (comment)

在正式写代码之前可以讨论下你的改进思路和原因，看下是否合理 。

问题就指派给你了 @ildnyy

@578223592 578223592 added the working 进行中 label Feb 5, 2024
@578223592
Copy link
Collaborator Author

@ildnyy hi，请问完成情况如何，如果有困难无法完成或者其他情况欢迎跟我沟通，如果你近期没有回复我，我可能会自己完成这个issue

@ildnyy
Copy link
Contributor

ildnyy commented Feb 20, 2024

@ildnyy hi，请问完成情况如何，如果有困难无法完成或者其他情况欢迎跟我沟通，如果你近期没有回复我，我可能会自己完成这个issue

已经改了，但是不是很懂怎么突出测试，但是原本的都能运行

@578223592
Copy link
Collaborator Author

@ildnyy hi，请问完成情况如何，如果有困难无法完成或者其他情况欢迎跟我沟通，如果你近期没有回复我，我可能会自己完成这个issue

已经改了，但是不是很懂怎么突出测试，但是原本的都能运行

@ildnyy 你可以先提交pr，然后在pr里面说明一下你的主要改动和改动的原因（思路）

@578223592 578223592 mentioned this issue Feb 23, 2024
Merged
578223592 added a commit that referenced this issue Feb 23, 2024
@ildnyy @578223592
feat:RPC自定义协议的头部字段长度优化 (#32)
49e076e 
* feat:RPC自定义协议的头部字段长度优化

* doc and comment

#19
---------

Co-authored-by: siwuxie <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ildnyy ildnyy

Labels
enhancement New feature or request good first issue Good for newcomers working 进行中
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ildnyy @578223592