[SERVER-18283] Cleanup the sharding map/reduce implementation Created: 30/Apr/15 Updated: 06/Dec/22 Resolved: 16/Dec/19 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | MapReduce, Sharding |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Improvement | Priority: | Major - P3 |
| Reporter: | Kaloian Manassiev | Assignee: | [DO NOT USE] Backlog - Sharding Team |
| Resolution: | Duplicate | Votes: | 1 |
| Labels: | FewerMongosPipelines | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||
| Assigned Teams: |
Sharding
|
||||||||||||
| Participants: | |||||||||||||
| Description |
|
The MongoS implementation of map/reduce should be moved to a separate file and it should also be split into smaller stages from the huge monolithic piece of code that it currently is. |
| Comments |
| Comment by Nicholas Zolnierz [ 14/Oct/19 ] | ||||||||||||||||||||||
|
This will go away as part of PM-766. | ||||||||||||||||||||||
| Comment by Scott Hernandez (Inactive) [ 05/May/15 ] | ||||||||||||||||||||||
|
As discussed in person, we should fix-up things to be consistent and use "long long" everywhere in our code. So from the commented code above we can do this:
and store Config::splitInfo as "long long" as well. | ||||||||||||||||||||||
| Comment by Githook User [ 04/May/15 ] | ||||||||||||||||||||||
|
Author: {u'username': u'kaloianm', u'name': u'Kaloian Manassiev', u'email': u'kaloian.manassiev@mongodb.com'}Message: | ||||||||||||||||||||||
| Comment by Githook User [ 04/May/15 ] | ||||||||||||||||||||||
|
Author: {u'username': u'kaloianm', u'name': u'Kaloian Manassiev', u'email': u'kaloian.manassiev@mongodb.com'}Message: | ||||||||||||||||||||||
| Comment by Scott Hernandez (Inactive) [ 04/May/15 ] | ||||||||||||||||||||||
|
Here is an example of the failure: https://evergreen.mongodb.com/task/mongodb_mongo_master_osx_108_d2db2f3827a762daf5b3af1611525f977ef1b551_15_05_04_14_10_08_sharding_osx_108 From: mr.cpp line 270 –
| ||||||||||||||||||||||
| Comment by Githook User [ 04/May/15 ] | ||||||||||||||||||||||
|
Author: {u'username': u'kaloianm', u'name': u'Kaloian Manassiev', u'email': u'kaloian.manassiev@mongodb.com'}Message: Revert " This reverts commit d2db2f3827a762daf5b3af1611525f977ef1b551. | ||||||||||||||||||||||
| Comment by Githook User [ 04/May/15 ] | ||||||||||||||||||||||
|
Author: {u'username': u'kaloianm', u'name': u'Kaloian Manassiev', u'email': u'kaloian.manassiev@mongodb.com'}Message: This change moves the sharding implementation of these two commands to |