[SERVER-17275] Remove "stats" field from distinct command response Created: 13/Feb/15 Updated: 19/Jul/17 Resolved: 15/Jul/16 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Diagnostics, Querying |
| Affects Version/s: | 3.0.0-rc8 |
| Fix Version/s: | 3.3.10 |
| Type: | Improvement | Priority: | Major - P3 |
| Reporter: | Jason R. Coombs | Assignee: | James Wahlin |
| Resolution: | Done | Votes: | 1 |
| Labels: | neweng | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||||||
| Backwards Compatibility: | Fully Compatible | ||||||||||||||||||||
| Sprint: | Integration 17 (07/15/16) | ||||||||||||||||||||
| Participants: | |||||||||||||||||||||
| Description |
|
When running a distinct query, the 'stats' still return using the keys from MongoDB 2.x:
Since the explain output has renamed those stats keys' names, perhaps the distinct query should do the same for consistency. Specifically, those keys should be nReturned, totalKeysExamined, and totalDocsExamined. Best to do that before 3.0 goes gold, as it'll be more difficult and clumsy to perform later. |
| Comments |
| Comment by Githook User [ 15/Jul/16 ] |
|
Author: {u'username': u'jameswahlin', u'name': u'James Wahlin', u'email': u'james.wahlin@10gen.com'}Message: |
| Comment by Kyle Suarez [ 23/Mar/16 ] |
|
Awesome, thanks! I'll fix the relevant DOCS links. |
| Comment by James Wahlin [ 23/Mar/16 ] |
|
kyle.suarez - I spun off your request for field renames to SERVER-23325. This ticket will focus exclusively on retiring the distinct command "stats" field. |
| Comment by J Rassi [ 01/Mar/16 ] |
|
I'm repurposing this ticket to remove the "stats" field entirely from the command response, as this information is now available from the explain command. |
| Comment by Kyle Suarez [ 09/Dec/15 ] |
|
Documentation on |
| Comment by Kyle Suarez [ 09/Dec/15 ] |
|
In addition, the following stats use legacy names and should be changed for consistency with both explain() and profiler output:
|