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

Replace incorrect use of getStatusFromCommandResult in ShardServerCatalogCacheLoader

    • Type: Icon: Bug Bug
    • Resolution: Fixed
    • Priority: Icon: Major - P3 Major - P3
    • 3.5.9
    • Affects Version/s: None
    • Component/s: Sharding
    • None
    • Fully Compatible
    • ALL
    • Sharding 2017-05-29, Sharding 2017-06-19
    • 0

      The shard chunk loader is using getStatusFromCommandResult to parse command responses. However, getStatusFromCommandResult does not parse writeErrors and should not be used to parse write command response. The code should be moved off of getStatusFromCommandResult and instead use the BatchedCommandResponse::toStatus() function.

      Additionally, add a note to the getStatusFromCommandResult function comment that it cannot be used for write command responses.

            Assignee:
            dianna.hohensee@mongodb.com Dianna Hohensee (Inactive)
            Reporter:
            dianna.hohensee@mongodb.com Dianna Hohensee (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: