loadLastOpTimeAndWallTime should use IDL to parse oplog entries

XMLWordPrintableJSON

    • Type: Bug
    • Resolution: Fixed
    • Priority: Major - P3
    • 4.3.1
    • Affects Version/s: None
    • Component/s: Replication
    • None
    • Fully Compatible
    • ALL
    • v4.2, v4.0, v3.6
    • Repl 2019-08-12, Repl 2019-08-26, Repl 2019-09-09, Repl 2019-09-23
    • None
    • 3
    • None
    • None
    • None
    • None
    • None
    • None

      As of SERVER-40080 here the "wall" field of oplog entries no longer became optional like it is in the IDL. If we used the IDL for parsing the oplog entry there, then this backwards breaking change wouldn't have happened since the IDL considers it optional. This actually should be safe since we've been adding 'wall' since 3.6 and it's about time we made it mandatory, but we should do that in the IDL, not in one particular place where we parse the oplog entry individually.

      Specifically, we should use the OplogEntry class to parse oplog entries everywhere.

            Assignee:
            Xuerui Fa
            Reporter:
            Judah Schvimer
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

              Created:
              Updated:
              Resolved: