Uploaded image for project: 'Documentation'
  1. Documentation
  2. DOCS-13838

Investigate changes in SERVER-49392: Make storage engine init check that enableMajorityReadConcern is compatible with supportsReadConcernMajority

    XMLWordPrintable

    Details

    • Type: Task
    • Status: Closed
    • Priority: Major - P3
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 4.7.0
    • Component/s: None
    • Labels:
      None

      Description

      Description

      Downstream Change Summary

      Storage engines that do not support majority read concern but are started with the --enableMajorityReadConcern option will fail to startup with an error message.

      See this test for the error message: https://github.com/mongodb/mongo/blob/56b83070a83b2bf134a54063b27c2b48b22dd852/jstests/noPassthrough/supports_read_concern_majority.js

      Description of Linked Ticket

      The enableMajorityReadConcern server parameter defaults to true, even when using a storage engine where supportsReadConcernMajority() is false. Ideally it should default to false in such cases, however, implementing this is probably more effort than it's worth (it only affects test storage engines). Instead, storage engine creation/init should check if supportsReadConcernMajority is false and enableMajorityReadConcern is true, and handle it accordingly: either force enableMajorityReadConcern to false (if it isn't too late to do that), or else abort startup with an error message (saying that this combination isn't valid and enableMajorityReadConcern must be disabled to run with this storage engine).

      Scope of changes

      Impact to Other Docs

      MVP (Work and Date)

      Resources (Scope or Design Docs, Invision, etc.)

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              ian.fogelman Ian Fogelman
              Reporter:
              backlog-server-pm Backlog - Core Eng Program Management Team
              Participants:
              Last commenter:
              Ian Fogelman Ian Fogelman
              External Reviewer:
              Gregory Wlodarek
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Days since reply:
                6 weeks, 6 days ago
                Date of 1st Reply: