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

Feature/txn (#1585) #2124

Merged
merged 5 commits into from
Nov 21, 2023
Merged

Feature/txn (#1585) #2124

merged 5 commits into from
Nov 21, 2023

Conversation

chejinge
Copy link
Collaborator

@chejinge chejinge commented Nov 17, 2023

ForestLH and others added 2 commits September 21, 2023 11:33
* fix: fix select cmd return inconsistent with redis

Signed-off-by: Hao Lee <1838249551@qq.com>

* refactor:modified lock style while involve db level

Signed-off-by: Hao Lee <1838249551@qq.com>

* feature:txn basic

Signed-off-by: Hao Lee <1838249551@qq.com>

* fix:merge upstream

Signed-off-by: Hao Lee <1838249551@qq.com>

* feature:txn udpate

Signed-off-by: Hao Lee <1838249551@qq.com>

* feature:add txn for pika(#1446)

todo:test txn. Just to verify the feasibility of the program.
Signed-off-by: Hao Lee <1838249551@qq.com>

* update unwatch cmd

Add comments

Signed-off-by: Hao Lee <1838249551@qq.com>

* clear watched key when connection closed

Signed-off-by: Hao Lee <1838249551@qq.com>

* merge upstream code

Signed-off-by: Hao Lee <1838249551@qq.com>

* update

Signed-off-by: Hao Lee <1838249551@qq.com>

* feature: add txn for pika completely

Signed-off-by: Hao Lee <1838249551@qq.com>

* add set txn failed for modified watch key

Signed-off-by: Hao Lee <1838249551@qq.com>

* update:reduce the particle size of the lock in txn

Signed-off-by: Hao Lee <1838249551@qq.com>

* chore:remove redundant comment

Signed-off-by: Hao Lee <1838249551@qq.com>

* test:add go ci test for txn

Signed-off-by: Hao Lee <1838249551@qq.com>

* fix compile error for linux

Signed-off-by: Hao Lee <1838249551@qq.com>

* update txn go ci test

Signed-off-by: Hao Lee <1838249551@qq.com>

* update txn for block list pop command

Signed-off-by: Hao Lee <1838249551@qq.com>

* Improve blpop-related in Redis transactions

Signed-off-by: Hao Lee <1838249551@qq.com>

* blpop_txn_fix

* add some test for go test txn

Signed-off-by: Hao Lee <1838249551@qq.com>

* update txn integration test

Signed-off-by: Hao Lee <1838249551@qq.com>

* txn change class to struct

Signed-off-by: Hao Lee <1838249551@qq.com>

* txn:use weak ptr instead of shared ptr in Cmd

Signed-off-by: Hao Lee <1838249551@qq.com>

---------

Signed-off-by: Hao Lee <1838249551@qq.com>
Co-authored-by: cheniujh <1271435567@qq.com>
@chejinge chejinge closed this Nov 17, 2023
@chejinge chejinge reopened this Nov 17, 2023
include/pika_command.h Outdated Show resolved Hide resolved
ForestLH and others added 2 commits November 20, 2023 19:22
Signed-off-by: LeeHao <1838249551@qq.com>
* using func instead of class private member
AlexStocks
AlexStocks previously approved these changes Nov 21, 2023
@chejinge chejinge dismissed AlexStocks’s stale review November 21, 2023 02:39

The merge-base changed after approval.

@AlexStocks AlexStocks merged commit f1965de into unstable Nov 21, 2023
11 checks passed
@AlexStocks AlexStocks deleted the OpenAtom-Transaction branch November 21, 2023 02:41
bigdaronlee163 pushed a commit to bigdaronlee163/pika that referenced this pull request Jun 8, 2024
* Feature/txn (OpenAtomFoundation#1585)

* fix: fix select cmd return inconsistent with redis

Signed-off-by: Hao Lee <1838249551@qq.com>

* refactor:modified lock style while involve db level

Signed-off-by: Hao Lee <1838249551@qq.com>

* feature:txn basic

Signed-off-by: Hao Lee <1838249551@qq.com>

* fix:merge upstream

Signed-off-by: Hao Lee <1838249551@qq.com>

* feature:txn udpate

Signed-off-by: Hao Lee <1838249551@qq.com>

* feature:add txn for pika(OpenAtomFoundation#1446)

todo:test txn. Just to verify the feasibility of the program.
Signed-off-by: Hao Lee <1838249551@qq.com>

* update unwatch cmd

Add comments

Signed-off-by: Hao Lee <1838249551@qq.com>

* clear watched key when connection closed

Signed-off-by: Hao Lee <1838249551@qq.com>

* merge upstream code

Signed-off-by: Hao Lee <1838249551@qq.com>

* update

Signed-off-by: Hao Lee <1838249551@qq.com>

* feature: add txn for pika completely

Signed-off-by: Hao Lee <1838249551@qq.com>

* add set txn failed for modified watch key

Signed-off-by: Hao Lee <1838249551@qq.com>

* update:reduce the particle size of the lock in txn

Signed-off-by: Hao Lee <1838249551@qq.com>

* chore:remove redundant comment

Signed-off-by: Hao Lee <1838249551@qq.com>

* test:add go ci test for txn

Signed-off-by: Hao Lee <1838249551@qq.com>

* fix compile error for linux

Signed-off-by: Hao Lee <1838249551@qq.com>

* update txn go ci test

Signed-off-by: Hao Lee <1838249551@qq.com>

* update txn for block list pop command

Signed-off-by: Hao Lee <1838249551@qq.com>

* Improve blpop-related in Redis transactions

Signed-off-by: Hao Lee <1838249551@qq.com>

* blpop_txn_fix

* add some test for go test txn

Signed-off-by: Hao Lee <1838249551@qq.com>

* update txn integration test

Signed-off-by: Hao Lee <1838249551@qq.com>

* txn change class to struct

Signed-off-by: Hao Lee <1838249551@qq.com>

* txn:use weak ptr instead of shared ptr in Cmd

Signed-off-by: Hao Lee <1838249551@qq.com>

---------

Signed-off-by: Hao Lee <1838249551@qq.com>
Co-authored-by: cheniujh <1271435567@qq.com>

* FNT

* fix:txn compile error in ubuntu (OpenAtomFoundation#2128)

Signed-off-by: LeeHao <1838249551@qq.com>

* using func instead of class private member (OpenAtomFoundation#2130)

* using func instead of class private member

---------

Signed-off-by: Hao Lee <1838249551@qq.com>
Signed-off-by: LeeHao <1838249551@qq.com>
Co-authored-by: LeeHao <39085999+ForestLH@users.noreply.github.com>
Co-authored-by: cheniujh <1271435567@qq.com>
Co-authored-by: Xin.Zh <dragoncharlie@foxmail.com>
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

3 participants