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: hash cmd (hset/hget, hmset/hmget, hgetall, hkeys) #57

Merged
merged 15 commits into from
Dec 10, 2023

Conversation

longfar-ncy
Copy link
Collaborator

@longfar-ncy longfar-ncy commented Nov 27, 2023

  • 添加hash命令hset/hget, hmset/hmget, hgetall, hkeys (HASHcommand develop #38)
  • 使用 ADD_COMMAND 宏,修改已有 kv 命令使用宏的方式添加命令

@github-actions github-actions bot added the ✏️ Feature New feature or request label Nov 27, 2023
AlexStocks
AlexStocks previously approved these changes Nov 28, 2023
src/cmd_hash.cc Outdated Show resolved Hide resolved
@longfar-ncy
Copy link
Collaborator Author

冲突已解决 @AlexStocks @lqxhub

src/cmd_hash.cc Outdated Show resolved Hide resolved
src/cmd_hash.cc Outdated Show resolved Hide resolved
src/cmd_hash.cc Outdated Show resolved Hide resolved
@longfar-ncy
Copy link
Collaborator Author

突然发现redis hset 和 hmset 返回结果不一样,前者有显示数量,后者只有ok。因此还是将 hset 和 hmset 分开处理,与redis保持一致

src/cmd_hash.cc Outdated Show resolved Hide resolved
Centurybbx
Centurybbx previously approved these changes Dec 4, 2023
AlexStocks
AlexStocks previously approved these changes Dec 4, 2023
@longfar-ncy
Copy link
Collaborator Author

冲突已解决

@AlexStocks AlexStocks merged commit 232e3a9 into OpenAtomFoundation:unstable Dec 10, 2023
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0.1.0 core ✏️ Feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants