-
Notifications
You must be signed in to change notification settings - Fork 0
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
Adding command BLMove #324
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just a couple of minor documentation comments
java/client/src/main/java/glide/api/commands/ListBaseCommands.java
Outdated
Show resolved
Hide resolved
java/client/src/main/java/glide/api/commands/ListBaseCommands.java
Outdated
Show resolved
Hide resolved
@@ -3832,6 +3833,44 @@ public T lmove( | |||
return getThis(); | |||
} | |||
|
|||
/** | |||
* Blocks the connection until it pops Atomically pops and removes the left/right-most element to |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
* Blocks the connection until it pops Atomically pops and removes the left/right-most element to | |
* Blocks the connection until it atomically pops and removes the left/right-most element to |
java/client/src/main/java/glide/api/models/BaseTransaction.java
Outdated
Show resolved
Hide resolved
note: BLMOVE is already considered for timeout |
so if we encounter a command that is blocking and not yet included in |
All existing at the moment commands with timout added to |
java/client/src/main/java/glide/api/commands/ListBaseCommands.java
Outdated
Show resolved
Hide resolved
java/client/src/main/java/glide/api/commands/ListBaseCommands.java
Outdated
Show resolved
Hide resolved
java/client/src/main/java/glide/api/models/BaseTransaction.java
Outdated
Show resolved
Hide resolved
java/integTest/src/test/java/glide/TransactionTestUtilities.java
Outdated
Show resolved
Hide resolved
* Adding command BLMove (#324) Co-authored-by: TJ Zhang <tj.zhang@improving.com>
* Adding command BLMove (#324) Co-authored-by: TJ Zhang <tj.zhang@improving.com>
No description provided.