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

Replace incorrect use of getStatusFromCommandResult in ShardServerCatalogCacheLoader

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major - P3
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 3.5.9
    • Component/s: Sharding
    • Labels:
      None
    • Backwards Compatibility:
      Fully Compatible
    • Operating System:
      ALL
    • Sprint:
      Sharding 2017-05-29, Sharding 2017-06-19
    • Linked BF Score:
      0

      Description

      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.

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              dianna.hohensee Dianna Hohensee
              Reporter:
              dianna.hohensee Dianna Hohensee
              Participants:
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: