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 <[email protected]>

* refactor:modified lock style while involve db level

Signed-off-by: Hao Lee <[email protected]>

* feature:txn basic

Signed-off-by: Hao Lee <[email protected]>

* fix:merge upstream

Signed-off-by: Hao Lee <[email protected]>

* feature:txn udpate

Signed-off-by: Hao Lee <[email protected]>

* feature:add txn for pika(#1446)

todo:test txn. Just to verify the feasibility of the program.
Signed-off-by: Hao Lee <[email protected]>

* update unwatch cmd

Add comments

Signed-off-by: Hao Lee <[email protected]>

* clear watched key when connection closed

Signed-off-by: Hao Lee <[email protected]>

* merge upstream code

Signed-off-by: Hao Lee <[email protected]>

* update

Signed-off-by: Hao Lee <[email protected]>

* feature: add txn for pika completely

Signed-off-by: Hao Lee <[email protected]>

* add set txn failed for modified watch key

Signed-off-by: Hao Lee <[email protected]>

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

Signed-off-by: Hao Lee <[email protected]>

* chore:remove redundant comment

Signed-off-by: Hao Lee <[email protected]>

* test:add go ci test for txn

Signed-off-by: Hao Lee <[email protected]>

* fix compile error for linux

Signed-off-by: Hao Lee <[email protected]>

* update txn go ci test

Signed-off-by: Hao Lee <[email protected]>

* update txn for block list pop command

Signed-off-by: Hao Lee <[email protected]>

* Improve blpop-related in Redis transactions

Signed-off-by: Hao Lee <[email protected]>

* blpop_txn_fix

* add some test for go test txn

Signed-off-by: Hao Lee <[email protected]>

* update txn integration test

Signed-off-by: Hao Lee <[email protected]>

* txn change class to struct

Signed-off-by: Hao Lee <[email protected]>

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

Signed-off-by: Hao Lee <[email protected]>

---------

Signed-off-by: Hao Lee <[email protected]>
Co-authored-by: cheniujh <[email protected]>
@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
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 <[email protected]>

* refactor:modified lock style while involve db level

Signed-off-by: Hao Lee <[email protected]>

* feature:txn basic

Signed-off-by: Hao Lee <[email protected]>

* fix:merge upstream

Signed-off-by: Hao Lee <[email protected]>

* feature:txn udpate

Signed-off-by: Hao Lee <[email protected]>

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

todo:test txn. Just to verify the feasibility of the program.
Signed-off-by: Hao Lee <[email protected]>

* update unwatch cmd

Add comments

Signed-off-by: Hao Lee <[email protected]>

* clear watched key when connection closed

Signed-off-by: Hao Lee <[email protected]>

* merge upstream code

Signed-off-by: Hao Lee <[email protected]>

* update

Signed-off-by: Hao Lee <[email protected]>

* feature: add txn for pika completely

Signed-off-by: Hao Lee <[email protected]>

* add set txn failed for modified watch key

Signed-off-by: Hao Lee <[email protected]>

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

Signed-off-by: Hao Lee <[email protected]>

* chore:remove redundant comment

Signed-off-by: Hao Lee <[email protected]>

* test:add go ci test for txn

Signed-off-by: Hao Lee <[email protected]>

* fix compile error for linux

Signed-off-by: Hao Lee <[email protected]>

* update txn go ci test

Signed-off-by: Hao Lee <[email protected]>

* update txn for block list pop command

Signed-off-by: Hao Lee <[email protected]>

* Improve blpop-related in Redis transactions

Signed-off-by: Hao Lee <[email protected]>

* blpop_txn_fix

* add some test for go test txn

Signed-off-by: Hao Lee <[email protected]>

* update txn integration test

Signed-off-by: Hao Lee <[email protected]>

* txn change class to struct

Signed-off-by: Hao Lee <[email protected]>

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

Signed-off-by: Hao Lee <[email protected]>

---------

Signed-off-by: Hao Lee <[email protected]>
Co-authored-by: cheniujh <[email protected]>

* FNT

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

Signed-off-by: LeeHao <[email protected]>

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

* using func instead of class private member

---------

Signed-off-by: Hao Lee <[email protected]>
Signed-off-by: LeeHao <[email protected]>
Co-authored-by: LeeHao <[email protected]>
Co-authored-by: cheniujh <[email protected]>
Co-authored-by: Xin.Zh <[email protected]>
cheniujh added a commit to cheniujh/pika that referenced this pull request Sep 24, 2024
* Feature/txn (OpenAtomFoundation#1585)

* fix: fix select cmd return inconsistent with redis

Signed-off-by: Hao Lee <[email protected]>

* refactor:modified lock style while involve db level

Signed-off-by: Hao Lee <[email protected]>

* feature:txn basic

Signed-off-by: Hao Lee <[email protected]>

* fix:merge upstream

Signed-off-by: Hao Lee <[email protected]>

* feature:txn udpate

Signed-off-by: Hao Lee <[email protected]>

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

todo:test txn. Just to verify the feasibility of the program.
Signed-off-by: Hao Lee <[email protected]>

* update unwatch cmd

Add comments

Signed-off-by: Hao Lee <[email protected]>

* clear watched key when connection closed

Signed-off-by: Hao Lee <[email protected]>

* merge upstream code

Signed-off-by: Hao Lee <[email protected]>

* update

Signed-off-by: Hao Lee <[email protected]>

* feature: add txn for pika completely

Signed-off-by: Hao Lee <[email protected]>

* add set txn failed for modified watch key

Signed-off-by: Hao Lee <[email protected]>

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

Signed-off-by: Hao Lee <[email protected]>

* chore:remove redundant comment

Signed-off-by: Hao Lee <[email protected]>

* test:add go ci test for txn

Signed-off-by: Hao Lee <[email protected]>

* fix compile error for linux

Signed-off-by: Hao Lee <[email protected]>

* update txn go ci test

Signed-off-by: Hao Lee <[email protected]>

* update txn for block list pop command

Signed-off-by: Hao Lee <[email protected]>

* Improve blpop-related in Redis transactions

Signed-off-by: Hao Lee <[email protected]>

* blpop_txn_fix

* add some test for go test txn

Signed-off-by: Hao Lee <[email protected]>

* update txn integration test

Signed-off-by: Hao Lee <[email protected]>

* txn change class to struct

Signed-off-by: Hao Lee <[email protected]>

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

Signed-off-by: Hao Lee <[email protected]>

---------

Signed-off-by: Hao Lee <[email protected]>
Co-authored-by: cheniujh <[email protected]>

* FNT

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

Signed-off-by: LeeHao <[email protected]>

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

* using func instead of class private member

---------

Signed-off-by: Hao Lee <[email protected]>
Signed-off-by: LeeHao <[email protected]>
Co-authored-by: LeeHao <[email protected]>
Co-authored-by: cheniujh <[email protected]>
Co-authored-by: Xin.Zh <[email protected]>
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