-
Notifications
You must be signed in to change notification settings - Fork 4.1k
streamingest: TestRandomClientGeneration should use the proper SQLCodec in assertEqualKVs() #82883
Copy link
Copy link
Closed
Labels
A-multitenancyRelated to multi-tenancyRelated to multi-tenancyA-tenant-streamingIncluding cluster streamingIncluding cluster streamingA-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.
Milestone
Description
The test TestRandomClientGeneration in streamingest/stream_ingestion_processor_test.go currently uses keys.TODOSQLCodec() in its post-conditiopn assertEqualKVs().
Instead this should be modified to use the codec expected for the replication stream being validated.
cc @stevendanna for triage.
Jira issue: CRDB-16710
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
A-multitenancyRelated to multi-tenancyRelated to multi-tenancyA-tenant-streamingIncluding cluster streamingIncluding cluster streamingA-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.