[SERVER-57195] Convert resharding document deletion to use PersistentTaskStore instead of creating a WriteUnitOfWork Created: 25/May/21 Updated: 09/Jun/21 Resolved: 09/Jun/21 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Sharding |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Task | Priority: | Major - P3 |
| Reporter: | Blake Oler | Assignee: | Junhson Jean-Baptiste (Inactive) |
| Resolution: | Won't Do | Votes: | 0 |
| Labels: | PM-234-M3, PM-234-T-autocommits, neweng | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||||||
| Sprint: | Sharding 2021-06-14 | ||||||||||||||||||||
| Participants: | |||||||||||||||||||||
| Story Points: | 1 | ||||||||||||||||||||
| Description |
|
We changed the document deletion to use a manual WriteUnitOfWork, since we needed to leverage the onCommit handler to fulfill the completion promise. This was a temporary solution to As code cleanup we can convert resharding document deletion back to using the PersistentTaskStore, which uses fewer lines of code. |
| Comments |
| Comment by Janna Golden [ 09/Jun/21 ] |
|
Closing in favor of |
| Comment by Haley Connelly [ 09/Jun/21 ] |
|
After chatting with max.hirschhorn about |