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

Java: Add UNLINK() command #976

Conversation

acarbonetto
Copy link
Collaborator

Issue #, if available:

Description of changes:

Adding the UNLINK() command (Generic command). Note, unlink() and del() are very similar commands.

Example:

// assuming key1 and key2 exist, while key "invalid" does not. 
Integer result = client.unlink(new String[] {"key1", "key2", "invalid"}).get();
assert result == 2;

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@acarbonetto acarbonetto requested a review from a team as a code owner February 15, 2024 23:23
@acarbonetto acarbonetto force-pushed the java/integ_acarbo_add_unlink_command branch from 6eebf95 to 26e7a0b Compare February 15, 2024 23:24
@acarbonetto acarbonetto reopened this Feb 16, 2024
@acarbonetto acarbonetto changed the title Java: Add UNLINK() command (#94) Java: Add UNLINK() command Feb 16, 2024
@Yury-Fridlyand Yury-Fridlyand added the java issues and fixes related to the java client label Feb 16, 2024
@acarbonetto acarbonetto force-pushed the java/integ_acarbo_add_unlink_command branch from 076b683 to 699da14 Compare February 16, 2024 20:30
Copy link
Collaborator

@Yury-Fridlyand Yury-Fridlyand left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

rebase?

@acarbonetto acarbonetto force-pushed the java/integ_acarbo_add_unlink_command branch from 0145826 to 99dd85a Compare February 21, 2024 02:29
Copy link
Collaborator

@Yury-Fridlyand Yury-Fridlyand left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

More rebase for the god of rebase?

acarbonetto and others added 5 commits February 22, 2024 14:02
Signed-off-by: Andrew Carbonetto <andrew.carbonetto@improving.com>
Signed-off-by: Andrew Carbonetto <andrew.carbonetto@improving.com>
Signed-off-by: Andrew Carbonetto <andrew.carbonetto@improving.com>
Signed-off-by: Andrew Carbonetto <andrew.carbonetto@improving.com>
@SanHalacogluImproving SanHalacogluImproving merged commit d2d6b2f into valkey-io:main Feb 22, 2024
11 checks passed
@SanHalacogluImproving SanHalacogluImproving deleted the java/integ_acarbo_add_unlink_command branch March 19, 2024 18:02
cyip10 pushed a commit to Bit-Quill/valkey-glide that referenced this pull request Jun 24, 2024
* Java: Add Unlink command back after merge

Signed-off-by: Andrew Carbonetto <andrew.carbonetto@improving.com>

* Doc fixes

Signed-off-by: Andrew Carbonetto <andrew.carbonetto@improving.com>

* Update unlink for review comments

Signed-off-by: Andrew Carbonetto <andrew.carbonetto@improving.com>

* Fix merge conflicts

Signed-off-by: Andrew Carbonetto <andrew.carbonetto@improving.com>

* Minor documentation update to redisClientTest.

* Added unlink test to transactions.

---------

Signed-off-by: Andrew Carbonetto <andrew.carbonetto@improving.com>
Co-authored-by: SanHalacogluImproving <san.halacoglu@improving.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
java issues and fixes related to the java client
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants