[SERVER-55472] Make DBClientCursor::fromAggregationRequest return the original status rather than CommandFailed Created: 24/Mar/21 Updated: 06/Dec/22 Resolved: 25/Mar/21 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Replication |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Task | Priority: | Major - P3 |
| Reporter: | Cheahuychou Mao | Assignee: | Backlog - Replication Team |
| Resolution: | Duplicate | Votes: | 0 |
| Labels: | pm-1791_non-cloud-blocking | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||||||
| Assigned Teams: |
Replication
|
||||||||||||||||||||
| Participants: | |||||||||||||||||||||
| Description |
|
Currently, this method converts all non-OK status to CommandFailed and only returns the original status as the status reason. This is causing TenantMigrationRecipientService to fail to retry the aggregate command here on retryable errors. |
| Comments |
| Comment by Daniel Gottlieb (Inactive) [ 25/Mar/21 ] |
|
Hi cheahuychou.mao, does jordi.serra-torrens's patch address this for you? |