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

Remove OpObserverShardingImpl Dependency on OpObserverImpl

    XMLWordPrintableJSON

Details

    • Icon: Improvement Improvement
    • Resolution: Fixed
    • Icon: Major - P3 Major - P3
    • 7.1.0-rc0
    • None
    • None
    • None
    • Fully Compatible
    • Execution Team 2023-05-29, Execution Team 2023-06-12

    Description

      OpObserverShardingImpl inherits from OpObserverImpl, yet doesn't override any methods, it also has OpObserverImpl (its parent) aware of and calling into child methods. This breaks many aspects of good design and modularity.

       

      For sharded nodes, OpObserverImpl should be registered and perform its usual behavior explicitly.

      For the few actual dependencies on OpTime acquisition in OpObserverImpl, OpObserverImpl will call into OpObserverShardingImpl which will no longer be a child class.

      Attachments

        Activity

          People

            benety.goh@mongodb.com Benety Goh
            matt.kneiser@mongodb.com Matt Kneiser
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: