-
Notifications
You must be signed in to change notification settings - Fork 4.1k
colexecspan: TestSpanAssembler should exercise guest tenant keyspaces #82884
Copy link
Copy link
Closed
Labels
A-multitenancyRelated to multi-tenancyRelated to multi-tenancyA-testingTesting tools and infrastructureTesting tools and infrastructureC-bugCode not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior.Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior.E-quick-winLikely to be a quick win for someone experienced.Likely to be a quick win for someone experienced.T-sql-queriesSQL Queries TeamSQL Queries Team
Description
Forked from #48123.
Describe the problem
The test TestSpanAssembler in sql/colexec/colexecspan/span_assembler_test.go currently uses keys.TODOSQLCodec.
This means that it does not currently exercise secondary tenant keyspans.
Expected behavior
The test should be modified to contain two sub-tests, one using the system tenant codec and one for a secondary tenant.
cc @rharding6373 @yuzefovich for triage.
Jira issue: CRDB-16711
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
A-multitenancyRelated to multi-tenancyRelated to multi-tenancyA-testingTesting tools and infrastructureTesting tools and infrastructureC-bugCode not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior.Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior.E-quick-winLikely to be a quick win for someone experienced.Likely to be a quick win for someone experienced.T-sql-queriesSQL Queries TeamSQL Queries Team
Type
Projects
Status
Done