Consider adding a helper function to unify setting "real" and cached values in ReplicationCoordinatorImpl

XMLWordPrintableJSON

    • Type: Task
    • Resolution: Won't Do
    • Priority: Major - P3
    • None
    • Affects Version/s: None
    • Component/s: None
    • None
    • Replication
    • Repl 2026-02-02
    • None
    • None
    • None
    • None
    • None
    • None
    • None

      There are several variations of this:

      auto lastCommittedOpTimeAndWallTime = _topCoord->getLastCommittedOpTimeAndWallTime();
        _myLastCommittedOpTimeAndWallTimeCached.store(lk, lastCommittedOpTimeAndWallTime);

      It would be nice to find a pattern that updates both the "real" value (the private member) and also the cached value so that we don't forget to update the cached value.

            Assignee:
            Unassigned
            Reporter:
            Brad Cater
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: