-
Notifications
You must be signed in to change notification settings - Fork 4.1k
kvserver: investigate whether we can remove support for inline values from rangefeeds #69357
Copy link
Copy link
Closed
Labels
A-kv-replicationRelating to Raft, consensus, and coordination.Relating to Raft, consensus, and coordination.C-enhancementSolution expected to add code/behavior + preserve backward-compat (pg compat issues are exception)Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception)
Description
Currently, no users of rangefeeds should ever see an inline value as they all operate on normal SQL tables.
If we were to run rangefeed on range with inline values, we have at least a few problems:
- Since inline values do not have associated timestamps, we currently emit all inline values encountered during a catchup scan, even if they have not changed since the given cursor.
- If the inline value changed multiple times since the checkpoint, a catchup-scan will only observe the final value.
- If a TBI is used for the catchup scan we may miss the inline value completely.
Do we have future uses for rangefeeds over ranges with inline values?
Jira issue: CRDB-9556
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
A-kv-replicationRelating to Raft, consensus, and coordination.Relating to Raft, consensus, and coordination.C-enhancementSolution expected to add code/behavior + preserve backward-compat (pg compat issues are exception)Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception)