Skip to content

Multi-database support in cluster mode#242

Merged
zuiderkwast merged 8 commits into
valkey-io:mainfrom
xbasel:multidb
May 26, 2025
Merged

Multi-database support in cluster mode#242
zuiderkwast merged 8 commits into
valkey-io:mainfrom
xbasel:multidb

Conversation

@xbasel

@xbasel xbasel commented Mar 3, 2025

Copy link
Copy Markdown
Member
  • Updated documentation to reflect multi-database support in Valkey Cluster.
  • Removed outdated restrictions on SELECT and database-related commands.
  • Clarified that MIGRATE, COUNTKEYSINSLOT, and GETKEYSINSLOT operate within the selected database.
  • Clarified that SWAPDB is disabled in cluster mode to prevent shard inconsistencies.

- Updated documentation to reflect multi-database support in Valkey Cluster.
- Removed outdated restrictions on `SELECT` and database-related commands.
- Clarified that `MIGRATE`, `COUNTKEYSINSLOT`, and `GETKEYSINSLOT` operate within the selected database.
- Clarified that `SWAPDB` is disabled in cluster mode to prevent shard inconsistencies.

Signed-off-by: xbasel <103044017+xbasel@users.noreply.github.com>
@xbasel xbasel self-assigned this Mar 3, 2025
@xbasel xbasel added the documentation Improvements or additions to documentation label Mar 3, 2025
@xbasel

xbasel commented Mar 3, 2025

Copy link
Copy Markdown
Member Author

Depends on valkey-io/valkey#1671
Fixes: valkey-io/valkey#1319

@madolson madolson left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Mostly looks good, minor suggestions.

Comment thread commands/cluster-countkeysinslot.md Outdated
Comment thread commands/cluster-getkeysinslot.md
Comment thread commands/select.md Outdated
Comment thread topics/cluster-spec.md Outdated
Comment thread topics/cluster-spec.md Outdated
xbasel and others added 4 commits May 9, 2025 22:48
Co-authored-by: Madelyn Olson <madelyneolson@gmail.com>
Signed-off-by: xbasel <103044017+xbasel@users.noreply.github.com>
Co-authored-by: Madelyn Olson <madelyneolson@gmail.com>
Signed-off-by: xbasel <103044017+xbasel@users.noreply.github.com>
Co-authored-by: Madelyn Olson <madelyneolson@gmail.com>
Signed-off-by: xbasel <103044017+xbasel@users.noreply.github.com>
Signed-off-by: xbasel <103044017+xbasel@users.noreply.github.com>

@zuiderkwast zuiderkwast left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Genarally LGTM

Comment thread commands/swapdb.md Outdated
Comment thread topics/cluster-spec.md Outdated
Comment thread commands/select.md Outdated
@xbasel xbasel requested a review from zuiderkwast May 26, 2025 09:29
Signed-off-by: xbasel <103044017+xbasel@users.noreply.github.com>
xbasel added 2 commits May 26, 2025 13:29
Signed-off-by: xbasel <103044017+xbasel@users.noreply.github.com>
This reverts commit 43f16ec.

Signed-off-by: xbasel <103044017+xbasel@users.noreply.github.com>
@zuiderkwast zuiderkwast merged commit 8f9f504 into valkey-io:main May 26, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants