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

Move all unrelated work out of OpObserverImpl into a miscellaneous FallbackOpObserver

    XMLWordPrintableJSON

Details

    • Icon: Improvement Improvement
    • Resolution: Fixed
    • Icon: Major - P3 Major - P3
    • 7.1.0-rc0
    • None
    • None
    • None
    • Minor Change
    • Execution Team 2023-05-15

    Description

      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?

      Attachments

        Activity

          People

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

            Dates

              Created:
              Updated:
              Resolved: