You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@kvrocks.apache.org by "tarvfx (via GitHub)" <gi...@apache.org> on 2023/02/23 03:22:02 UTC

[GitHub] [incubator-kvrocks] tarvfx opened a new issue, #1277: Enable transaction support for the mset command

tarvfx opened a new issue, #1277:
URL: https://github.com/apache/incubator-kvrocks/issues/1277

   ### Search before asking
   
   - [X] I had searched in the [issues](https://github.com/apache/incubator-kvrocks/issues) and found no similar issues.
   
   
   ### Motivation
   
   Hi, guys
   
   Currently, using the mset command to write data in batches cannot ensure atomicity. We should let the system perform a rollback when the process is interrupted during data writing.
   
   Regard
   
   ### Solution
   
   _No response_
   
   ### Are you willing to submit a PR?
   
   - [ ] I'm willing to submit a PR!


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@kvrocks.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-kvrocks] PragmaTwice commented on issue #1277: Enable transaction support for the mset command

Posted by "PragmaTwice (via GitHub)" <gi...@apache.org>.
PragmaTwice commented on issue #1277:
URL: https://github.com/apache/incubator-kvrocks/issues/1277#issuecomment-1443975418

   Rather than making `mset` atomic, I wonder if it is better to add a new flag to `mset` for atomicity and leave the default behavior align with Redis.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@kvrocks.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-kvrocks] tarvfx commented on issue #1277: Enable transaction support for the mset command

Posted by "tarvfx (via GitHub)" <gi...@apache.org>.
tarvfx commented on issue #1277:
URL: https://github.com/apache/incubator-kvrocks/issues/1277#issuecomment-1443971071

   @git-hulk Thanks for you adding this feature to the to do list. In addition, I would like to add that other batch operation instructions other than mset, such as `msetnx`, should all be applicable to this enhanced feature.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@kvrocks.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [kvrocks] git-hulk commented on issue #1277: Enable transaction support for the mset command

Posted by "git-hulk (via GitHub)" <gi...@apache.org>.
git-hulk commented on issue #1277:
URL: https://github.com/apache/kvrocks/issues/1277#issuecomment-1680064333

   This was resolved in #1634 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@kvrocks.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [kvrocks] git-hulk closed issue #1277: Enable transaction support for the mset command

Posted by "git-hulk (via GitHub)" <gi...@apache.org>.
git-hulk closed issue #1277: Enable transaction support for the mset command
URL: https://github.com/apache/kvrocks/issues/1277


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@kvrocks.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-kvrocks] git-hulk commented on issue #1277: Enable transaction support for the mset command

Posted by "git-hulk (via GitHub)" <gi...@apache.org>.
git-hulk commented on issue #1277:
URL: https://github.com/apache/incubator-kvrocks/issues/1277#issuecomment-1442692500

   Thanks for @tarvfx filing this issue.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@kvrocks.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-kvrocks] git-hulk commented on issue #1277: Enable transaction support for the mset command

Posted by "git-hulk (via GitHub)" <gi...@apache.org>.
git-hulk commented on issue #1277:
URL: https://github.com/apache/incubator-kvrocks/issues/1277#issuecomment-1444968528

   @PragmaTwice Yes, we won't expect to affect the default behavior of the mset command.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@kvrocks.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org