[SERVER-29333] Replace incorrect use of getStatusFromCommandResult in ShardServerCatalogCacheLoader Created: 23/May/17 Updated: 30/Oct/23 Resolved: 02/Jun/17 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Sharding |
| Affects Version/s: | None |
| Fix Version/s: | 3.5.9 |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Dianna Hohensee (Inactive) | Assignee: | Dianna Hohensee (Inactive) |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||
| Backwards Compatibility: | Fully Compatible | ||||||||||||
| Operating System: | ALL | ||||||||||||
| Sprint: | Sharding 2017-05-29, Sharding 2017-06-19 | ||||||||||||
| Participants: | |||||||||||||
| 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. |
| Comments |
| Comment by Githook User [ 02/Jun/17 ] |
|
Author: {u'username': u'DiannaHohensee', u'name': u'Dianna Hohensee', u'email': u'dianna.hohensee@10gen.com'}Message: |
| Comment by Dianna Hohensee (Inactive) [ 25/May/17 ] |
|
This should be committed after |