Split Valkey/Redis into key/value parent with asymmetric siblings#245
Open
brianamarie wants to merge 14 commits into
Open
Split Valkey/Redis into key/value parent with asymmetric siblings#245brianamarie wants to merge 14 commits into
brianamarie wants to merge 14 commits into
Conversation
Replace the combined products/valkey-redis module with products/key-value so shared workloads stay one family while Valkey leads on memory, governance, and HA, and Redis keeps stay-or-migrate transition messaging for Kyle review. Co-authored-by: Cursor <cursoragent@cursor.com>
Bring PR #186 buyer, memory, and cache-tier positioning into the asymmetric parent/Valkey/Redis modules and matching value-pillar use cases, without restoring the combined page. Co-authored-by: Cursor <cursoragent@cursor.com>
Contributor
New File Governance Check
Missing or insufficient required sections:
Copy and paste the scaffold below into your PR body, then replace placeholders with specific details: ## New markdown file gate (required only when adding new `.md` files)
### Existing files reviewed first
- List the specific files you reviewed and what each one does.
### Exact gap not covered by existing files
- Explain what is still missing after reviewing existing canonical files.
### Why this must be a new canonical file
- State why editing an existing file is not sufficient.
### Owner and maintenance plan
- Name the owner and how this file will stay accurate over time.
### Decomposition and propagation plan
- List related files and downstream assets that may need updates.Doc Coverage Check
Docs missing discoverability links:
Add at least one markdown link to each missing file from README.md or CONTRIBUTING.md. Manual waiver commands (maintainers)
Waiver JSON is stored in a hidden PR comment ( Triggered by |
Contributor
Messaging Impact Check
licensing-or-open-source-claims (BLOCKING)License and open source positioning changes are cross-cutting. Required review files:
Suggested additional scan:
offering-name-or-tier-change (WARN)Offering naming updates should stay aligned across offering pages. Required review files:
Suggested additional scan:
product-messaging-module-touch (WARN)Product messaging modules often need checks against company framing and shared pillars or offerings. Required review files:
Suggested additional scan:
Manual waiver commands (maintainers):
Waiver state is stored in Triggered by |
Contributor
Messaging Smart Suggestions
Only auto-apply suggestions at high confidence with reviewer approval. Triggered by |
brianamarie
commented
Jul 23, 2026
brianamarie
commented
Jul 23, 2026
brianamarie
commented
Jul 23, 2026
Contributor
Markdown hygiene auto-fixApplied safe markdown hygiene fixes and pushed one bot commit to this PR branch. Fixed files:
Triggered by |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
products/valkey-redismodule withproducts/key-value: shared parent plus asymmetric Valkey and Redis children (KvRocks/Proxy stubs only), per @stockholmuxConsiderations