[SERVER-13985] printShardingStatus uses group/JS Created: 19/May/14 Updated: 11/Jul/16 Resolved: 27/May/14 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Shell |
| Affects Version/s: | 2.6.1 |
| Fix Version/s: | 2.6.12, 2.7.2 |
| Type: | Bug | Priority: | Minor - P4 |
| Reporter: | Asya Kamsky | Assignee: | Ramon Fernandez Marina |
| Resolution: | Done | Votes: | 1 |
| Labels: | neweng | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||||||||||
| Backwards Compatibility: | Major Change | ||||||||||||||||||||||||
| Operating System: | ALL | ||||||||||||||||||||||||
| Backport Completed: | |||||||||||||||||||||||||
| Sprint: | Server 2.7.1, Server 2.7.2 | ||||||||||||||||||||||||
| Participants: | |||||||||||||||||||||||||
| Linked BF Score: | 0 | ||||||||||||||||||||||||
| Description |
|
sh.status() aka printShardingStatus still uses the old MR/group command
with aggregation readily available without JS, there is no reason to leave this using group and being broken for users who turn off server side scripting for security reasons. |
| Comments |
| Comment by Githook User [ 02/Sep/15 ] | ||||||||||||||||||||||||||||||||||||||||||||||||
|
Author: {u'username': u'Zarkantho', u'name': u'Shaun Verch', u'email': u'shaun.verch@mongodb.com'}Message: (cherry picked from commit 6511fae500f5d3c3205499d0b48c842c199222f4) | ||||||||||||||||||||||||||||||||||||||||||||||||
| Comment by Githook User [ 02/Sep/15 ] | ||||||||||||||||||||||||||||||||||||||||||||||||
|
Author: {u'username': u'Zarkantho', u'name': u'Shaun Verch', u'email': u'shaun.verch@mongodb.com'}Message: (cherry picked from commit df9cb2c79f98f5ab5471a95b36e7689600d686d2) | ||||||||||||||||||||||||||||||||||||||||||||||||
| Comment by Shaun Verch [ 28/May/14 ] | ||||||||||||||||||||||||||||||||||||||||||||||||
|
Confirmed the new shell works with both versions when they have "--noscripting" on.
| ||||||||||||||||||||||||||||||||||||||||||||||||
| Comment by Githook User [ 28/May/14 ] | ||||||||||||||||||||||||||||||||||||||||||||||||
|
Author: {u'name': u'Shaun Verch', u'email': u'shaun.verch@mongodb.com'}Message: | ||||||||||||||||||||||||||||||||||||||||||||||||
| Comment by Scott Hernandez (Inactive) [ 27/May/14 ] | ||||||||||||||||||||||||||||||||||||||||||||||||
|
Note, this will not work against a server without aggregation support, like pre-2.2. We could add fallback support if the command is not found. | ||||||||||||||||||||||||||||||||||||||||||||||||
| Comment by Githook User [ 27/May/14 ] | ||||||||||||||||||||||||||||||||||||||||||||||||
|
Author: {u'name': u'Shaun Verch', u'email': u'shaun.verch@mongodb.com'}Message: | ||||||||||||||||||||||||||||||||||||||||||||||||
| Comment by Asya Kamsky [ 19/May/14 ] | ||||||||||||||||||||||||||||||||||||||||||||||||
|
Here's the change:
I tested it. Should I make/attach it as a pull request? | ||||||||||||||||||||||||||||||||||||||||||||||||
| Comment by Asya Kamsky [ 19/May/14 ] | ||||||||||||||||||||||||||||||||||||||||||||||||
|
Looks like it's 100% equivalent to doing aggregation:
in 2.6 anyway. |