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

STRING command develop #31

Closed
Tracked by #30
lqxhub opened this issue Nov 7, 2023 · 7 comments
Closed
Tracked by #30

STRING command develop #31

lqxhub opened this issue Nov 7, 2023 · 7 comments

Comments

@lqxhub
Copy link
Collaborator

lqxhub commented Nov 7, 2023 •
edited
Loading

string 相关的命令开发

现在已经写了 SET 和 GET的简单实现,

pikiwidb/src/cmd_kv.h

Line 15 in e05878d

class GetCmd : public BaseCmd {

命令的具体实现逻辑可以参考 pika, pika 的string类型命令的文件在 pika_kv.cc中
https:/OpenAtomFoundation/pika/blob/bdf90e81dce4cb49650f1a0d801e1c4e691d692f/src/pika_kv.cc#L6
@lqxhub lqxhub mentioned this issue Nov 7, 2023
Open
2 tasks
@AlexStocks
Copy link
Contributor

Task List:

PCY:

strlen
mset
msetnx
setnx
setex

Henry:

psetex
getset
mget
append
bitcount

CSX:

bitop
getbit
setbit
incr
decr

QX:

incrby
incrbyfloat
decrby
getrange
setrange

@jettcc
Copy link
Collaborator

jettcc commented Nov 15, 2023

@lqxhub i want to join you

@lqxhub
Copy link
Collaborator Author

lqxhub commented Nov 15, 2023

@lqxhub i want to join you

incrby
incrbyfloat
decrby
getrange
setrange

这几个命令先给你来做吧, 有问题随时沟通

@jettcc
Copy link
Collaborator

jettcc commented Nov 15, 2023 •
edited
Loading

这几个命令先给你来做吧, 有问题随时沟通

ok 😄

@quzizii
Copy link

quzizii commented Nov 15, 2023

我想要参与命令开发，可以分几个吗

@jettcc jettcc mentioned this issue Nov 19, 2023
Merged
@callme-taota callme-taota mentioned this issue Nov 20, 2023
Merged
This was referenced Nov 28, 2023
Merged
Merged
This was referenced Dec 4, 2023
Merged
Closed
This was referenced Dec 13, 2023
Closed
Merged
@hahahashen
Copy link
Contributor

set命令的ex nx选项什么的是不是还没有实现呀

@Issues-translate-bot
Copy link

Bot detected the issue body's language is not English, translate it automatically.

Is the ex nx option of the set command not implemented yet?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
6 participants
@AlexStocks @lqxhub @jettcc @hahahashen @Issues-translate-bot @quzizii