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

getDBForReadMirroring to return a DatabaseName

    • Type: Icon: Task Task
    • Resolution: Fixed
    • Priority: Icon: Major - P3 Major - P3
    • 7.1.0-rc0
    • Affects Version/s: None
    • Component/s: None
    • Fully Compatible
    • Server Serverless 2023-08-21

      The method getDBForReadMirroring is passing a cmdObj that's provided from the `body` of the `request` that is parsed in the BasicCommandWithReplyBuilderInterface::Invocation.

      That logic is odd and should be simplified.

      We should see if we can simply return the _dbName directly since the cmdObj() parameter comes from the `request.body` which is used to create the DatabaseName internal member _dbName. 

      Part of this change also include changing the return type from std::string to DatabaseName.

            Assignee:
            mathis.bessa@mongodb.com Mathis Bessa (Inactive)
            Reporter:
            mathis.bessa@mongodb.com Mathis Bessa (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: