Replace incorrect use of getStatusFromCommandResult in ShardServerCatalogCacheLoader

XMLWordPrintableJSON

    • Type: Bug
    • Resolution: Fixed
    • Priority: 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
    • None
    • 3
    • None
    • None
    • None
    • None
    • None
    • None

      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 (Inactive)
            Reporter:
            Dianna Hohensee (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: