-
Notifications
You must be signed in to change notification settings - Fork 53
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
Node: Add XAUTOCLAIM
command.
#2108
Node: Add XAUTOCLAIM
command.
#2108
Conversation
Signed-off-by: Yury-Fridlyand <[email protected]>
node/src/BaseClient.ts
Outdated
* @param start - Filters the claimed entries to those that have an ID equal or greater than the | ||
* specified value. | ||
* @param count - (Optional) Limits the number of claimed entries to the specified value. | ||
* @returns An `array` containing the following elements: |
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.
* @returns An `array` containing the following elements: | |
* @returns An array containing the following elements: |
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.
Why?
@@ -1051,7 +1062,7 @@ export async function transactionTest( | |||
baseTransaction.bitpos(key17, 1); | |||
responseData.push(["bitpos(key17, 1)", 1]); | |||
|
|||
if (gte("6.0.0", version)) { | |||
if (gte(version, "6.0.0")) { |
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.
we don't support anything less than 6...
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.
At least we should properly check the version, otherwise it is never tested
…im-valkey-83 Signed-off-by: Yury-Fridlyand <[email protected]>
No description provided.