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

feat(server): 参考Thrift从0->1写RPC传输、编码、多路复用模块 #25

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

Trinoooo
Copy link
Owner

@Trinoooo Trinoooo commented Apr 6, 2024

No description provided.

@Trinoooo Trinoooo marked this pull request as draft April 6, 2024 14:06
Copy link

codecov bot commented Apr 6, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 70.53%. Comparing base (48d8cea) to head (492008f).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #25   +/-   ##
=======================================
  Coverage   70.53%   70.53%           
=======================================
  Files           3        3           
  Lines         733      733           
=======================================
  Hits          517      517           
  Misses        140      140           
  Partials       76       76           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Trinoooo
Copy link
Owner Author

Trinoooo commented Apr 6, 2024

@RocooHash Hi!好久不见
最近俺学习了下apache thrift的源码,不满足于阅读,于是尝试着手实现。后才发现阅读过程中许多细节无意间被略过了,阶段性回顾发现许多成长点,感叹没有白搓。
从清明开始才腾出手写工作外的代码,后续会抽时间继续实现剩下的很大部分。数据存储模块我打算先看看开源数据库的源码再仿照着实现一遍,相信会有很大成长(是的,数据存储打算暂时搁置一段时间)!
相比23年Q3我尝试从网络标准库手搓HTTP服务器,本次我收敛了些 —— 基于net标准库 和 有thrift源码和文档可参考,难度下降了一个数量级,自底向上分别照猫画虎实现TransportProtocolProcessor模块。当然也加了些自己的思考在里面(我忽略掉了Thrift的许多历史包袱),最近长起来的自信让我甚至想去尝试实现Thrift的元编程,当然这是后话了hhh。

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.

None yet

1 participant