Skip to content

[4/4 messagequeue] Add Vitess vschema and two-shard vtcombo suite - #696

Draft
behinddwalls wants to merge 1 commit into
preetam/mq-tenant-wiringfrom
preetam/mq-tenant-vitess
Draft

[4/4 messagequeue] Add Vitess vschema and two-shard vtcombo suite#696
behinddwalls wants to merge 1 commit into
preetam/mq-tenant-wiringfrom
preetam/mq-tenant-vitess

Conversation

@behinddwalls

@behinddwalls behinddwalls commented Sep 8, 2026

Copy link
Copy Markdown
Collaborator

Summary

Why?

The tenant column is the production vindex. Without a VSchema and a two-shard suite, the schema change is unproven on Vitess.

What?

  • Add production xxhash VSchema for every queue table keyed on tenant.
  • Add a vtcombo integration suite that publishes and consumes across two shards (receive loop bound to the test context).

Admin CLI tenant flags shipped with the schema PR because ctl/lib SQL is coupled to the new primary keys.

Test Plan

Vitess suite is Docker-based; run with:
./tool/bazel test //test/integration/extension/messagequeue/mysql/vitess:go_default_test --strategy=TestRunner=local

Issues

Stack

  1. [1/4 messagequeue] RFC: per-tenant sharding for the MySQL message queue #693
  2. [2/4 messagequeue] Shard MySQL queues by tenant identity #694
  3. [3/4 messagequeue] Wire MQ_TENANTS through services and tests #695
  4. @ [4/4 messagequeue] Add Vitess vschema and two-shard vtcombo suite #696

## Summary

### Why?

The tenant column is the production vindex. Without a VSchema and a two-shard suite, the schema change is unproven on Vitess.

### What?

- Add production xxhash VSchema for every queue table keyed on tenant.
- Add a vtcombo integration suite that publishes and consumes across two shards (receive loop bound to the test context).

Admin CLI tenant flags shipped with the schema PR because ctl/lib SQL is coupled to the new primary keys.

## Test Plan

Vitess suite is Docker-based; run with:
`./tool/bazel test //test/integration/extension/messagequeue/mysql/vitess:go_default_test --strategy=TestRunner=local`

Co-authored-by: Cursor <cursoragent@cursor.com>
@behinddwalls
behinddwalls force-pushed the preetam/mq-tenant-vitess branch from 10f0140 to 0b69e19 Compare September 8, 2026 06:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant