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: Allow cluster wide scan with not fully covered cluster #10

Open
wants to merge 9 commits into
base: Node@allow-ucslots-Cscan
Choose a base branch
from

Conversation

jamesx-improving
Copy link

This is a FOR REVIEW ONLY PR with the Java part for issue: valkey-io#2436

This will eventually merge to https://github.com/valkey-io/valkey-glide/tree/release-1.2

@avifenesh avifenesh force-pushed the Node@allow-ucslots-Cscan branch from a42b9ec to 2dec9d0 Compare December 18, 2024 17:20
@yipin-chen
Copy link

I only review Java portion and the changes look fine. Please resolve the conflicts.

@jamesx-improving jamesx-improving force-pushed the java/jamesx-allow-ucslots-scan branch from 978750a to d1a9be0 Compare December 19, 2024 00:25
@avifenesh avifenesh force-pushed the Node@allow-ucslots-Cscan branch 6 times, most recently from 57c2c6c to 6518cf3 Compare December 22, 2024 12:02
@yipin-chen
Copy link

yipin-chen commented Dec 23, 2024

@jamesx-improving, Avi has merged his changes. Please rebase your PR.
Do you have any update from your manual testing? Are we able to get this merge on Monday?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants