[SERVER-17051] Remove runCount() function Created: 26/Jan/15 Updated: 18/Sep/15 Resolved: 17/Feb/15 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Querying |
| Affects Version/s: | None |
| Fix Version/s: | 3.1.0 |
| Type: | Task | Priority: | Minor - P4 |
| Reporter: | David Storch | Assignee: | David Storch |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Backwards Compatibility: | Fully Compatible |
| Participants: |
| Description |
|
The count command path was refactored for version 3.0. As part of this refactor, the runCount() function is now used only by the DBDirectClient. We should change the DBDirectClient to use the regular count command, allowing us to delete runCount(). This way there will be a single code path for running counts, which will be easier to maintain. |
| Comments |
| Comment by Githook User [ 17/Feb/15 ] |
|
Author: {u'username': u'dstorch', u'name': u'David Storch', u'email': u'david.storch@10gen.com'}Message: This special count path was used only by the DBDirectClient and is no longer needed. |
| Comment by David Storch [ 11/Feb/15 ] |
|
Commit caused a test failure in sharding/read_pref_cmd.js. Reverting and reopening this ticket until I get a chance to track down the failure. |
| Comment by Githook User [ 11/Feb/15 ] |
|
Author: {u'username': u'dstorch', u'name': u'David Storch', u'email': u'david.storch@10gen.com'}Message: Revert " This reverts commit 40e5c7f39b967411bba4733c445f408f906a26b5. |
| Comment by Githook User [ 10/Feb/15 ] |
|
Author: {u'username': u'dstorch', u'name': u'David Storch', u'email': u'david.storch@10gen.com'}Message: This special count path was used only by the DBDirectClient and is no longer needed. |