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

Tair #637

Closed
wants to merge 5 commits into from
Closed

Tair #637

wants to merge 5 commits into from

Conversation

lyj199907
Copy link
Contributor

ospp : add the support for the parse of tairString, tairHash, tairZset

  • test for operationing mode
    [x] :test for restore
    [x] : test for sync
    [] : test for san

  • test for runing env
    [x] : standlone
    [] : cluster

@CLAassistant
Copy link

CLAassistant commented Jul 20, 2023

CLA assistant check
All committers have signed the CLA.

@suxb201
Copy link
Member

suxb201 commented Jul 28, 2023

哈喽,我看了一遍,应该没有问题。但是有几点需要改进一下:

  1. PR 的标题和内容可以规范下格式,标题要说明做了什么改动。本 Repo 的社区交流不强制使用英文,git commit msg 需要英文。
  2. 需要补充一下自动化测试,每个命令都需要涉及到来确保不会出问题。测试框架我可能下周一、二能修一下,到时候可以添加一些测试了。

@lyj199907
Copy link
Contributor Author

lyj199907 commented Jul 28, 2023 via email

@lyj199907
Copy link
Contributor Author

变基,新的 pr 已挪到新的分支上#643

@lyj199907 lyj199907 closed this Aug 6, 2023
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.

3 participants