-
Notifications
You must be signed in to change notification settings - Fork 4.1k
storage: pointSynthesizingIter should only synthesize points above existing points #83899
Copy link
Copy link
Closed
Labels
A-kv-replicationRelating to Raft, consensus, and coordination.Relating to Raft, consensus, and coordination.A-storageRelating to our storage engine (Pebble) on-disk storage.Relating to our storage engine (Pebble) on-disk storage.C-performancePerf of queries or internals. Solution not expected to change functional behavior.Perf of queries or internals. Solution not expected to change functional behavior.
Description
pointSynthesizingIter currently synthesizes points both above and below existing points. We could probably get away with only synthesizing them above existing points, which would also avoid additional reverse peeks in SeekGE` that are likely very slow.
NB: This sort of assumes that synthetic points are tombstones, since we may not want to skip other synthetic points. But we can cross that bridge when we get there.
Jira issue: CRDB-17346
Epic CRDB-2624
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
A-kv-replicationRelating to Raft, consensus, and coordination.Relating to Raft, consensus, and coordination.A-storageRelating to our storage engine (Pebble) on-disk storage.Relating to our storage engine (Pebble) on-disk storage.C-performancePerf of queries or internals. Solution not expected to change functional behavior.Perf of queries or internals. Solution not expected to change functional behavior.