[SERVER-25124] Remove recursion from the MigrationManager Created: 18/Jul/16 Updated: 10/Aug/16 Resolved: 09/Aug/16 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Sharding |
| Affects Version/s: | 3.3.9 |
| Fix Version/s: | None |
| Type: | Improvement | Priority: | Major - P3 |
| Reporter: | Kaloian Manassiev | Assignee: | Kaloian Manassiev |
| Resolution: | Duplicate | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Backwards Compatibility: | Fully Compatible | ||||||||
| Sprint: | Sharding 18 (08/05/16), Sharding 2016-08-29 | ||||||||
| Participants: | |||||||||
| Description |
|
The MigrationManager's _executeMigrations method is currently recursive with no limit on the depth of recursion. In practice, this recursion should never be more than a couple of levels deep, because there aren't that many migrations happening per round, but for the sake of having a bullet-proof code we should rewrite it to not use recursion. |
| Comments |
| Comment by Dianna Hohensee (Inactive) [ 09/Aug/16 ] |
|
|