Uploaded image for project: 'Core Server'
  1. Core Server
  2. SERVER-76268

Move all unrelated work out of OpObserverImpl into a miscellaneous FallbackOpObserver

    • Type: Icon: Improvement Improvement
    • Resolution: Fixed
    • Priority: Icon: Major - P3 Major - P3
    • 7.1.0-rc0
    • Affects Version/s: None
    • Component/s: None
    • None
    • Minor Change
    • Execution Team 2023-05-15

      OpObserverImpl has far too many dependencies that are unnecessary.

      There are many code snippets that have bloated this entity that should be mechanically separated out and put into other OpObserver(s). For the purposes of this ticket and breaking the immediate dependency on OpObserverImpl, I will move all this code into a catch-all OpObserver. A future author could consider breaking these up into more logical blocks.

      Primarily, OpObserverImpl should have no need to communicate with these entities:

      • Scope
      • MongoDSessionCatalog
      • ReadWriteConcernDefaults
      • LogicalTimeValidator?

            Assignee:
            matt.kneiser@mongodb.com Matt Kneiser
            Reporter:
            matt.kneiser@mongodb.com Matt Kneiser
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: