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

合并 memecache text/binary 协议 #79

Open
4 tasks
wayslog opened this issue Jul 9, 2019 · 0 comments
Open
4 tasks

合并 memecache text/binary 协议 #79

wayslog opened this issue Jul 9, 2019 · 0 comments
Labels
2.0 release overlord 2.0 release 相关

Comments

@wayslog
Copy link
Collaborator

wayslog commented Jul 9, 2019

memcache 协议现在各种客户端的实现参差不齐,binary和text都有在用。现在采取的办法是用两个不同的端口分别指定为不同的协议。但是其实这个协议可以通过parser detect的方式进行兼容。
需要修改的地方:

  • 修改 parser ,让 parser 能同时实例化两种请求
  • 修改 NodeConn ,让 read/write 的操作同时支持
  • 修改 pipeline 模式,让 setq 等更加简单
  • 加入单元测试,测试 text 和 binary 协议随机出现的结果
@wayslog wayslog added the 2.0 release overlord 2.0 release 相关 label Jul 9, 2019
@wayslog wayslog added this to the proxy 2.0 release milestone Jul 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2.0 release overlord 2.0 release 相关
Projects
None yet
Development

No branches or pull requests

1 participant