Refactor WTRecordStoreCursor timestamp fields

XMLWordPrintableJSON

    • Type: Improvement
    • Resolution: Gone away
    • Priority: Minor - P4
    • None
    • Affects Version/s: None
    • Component/s: None
    • Storage Execution
    • None
    • 3
    • None
    • None
    • None
    • None
    • None
    • None
    • None

      Oplog cursors keep around two timestamps as state. The traditional visibility "no holes" time and SERVER-63308 introduced _readTimestampForOplog.

      Given visibility should ultimately be the minimum of the two values (when they both exist), it doesn't seem necessary to have two variables. However, an initial effort to make that reduction failed.

      I speculate some of the complexity has to do with oplog visibility being reset on _txnClose, but the _timestampReadSource is not. It's unclear what callers rely on those semantics.

            Assignee:
            [DO NOT USE] Backlog - Storage Execution Team
            Reporter:
            Daniel Gottlieb (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: