-
Notifications
You must be signed in to change notification settings - Fork 4.1k
cdc: Re-enable fingerprint validator in Validators test #31110
Copy link
Copy link
Closed
Labels
C-cleanupTech debt, refactors, loose ends, etc. Solution not expected to significantly change behavior.Tech debt, refactors, loose ends, etc. Solution not expected to significantly change behavior.
Description
Validators test is temporarily being run without the fingerprint validator because of changes to support schema backfills in #30902. Specifically, the fingerprint validator previously expected to receive the entire history of the table in original timestamp order, but the initial scan now returns all values at the statement time.
Fingerprint validator will need to be re-worked to support the new behavior.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
C-cleanupTech debt, refactors, loose ends, etc. Solution not expected to significantly change behavior.Tech debt, refactors, loose ends, etc. Solution not expected to significantly change behavior.