[SERVER-6978] getParameter should know all setParameter params Created: 09/Sep/12 Updated: 10/Nov/15 Resolved: 14/Aug/13 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 2.5.2 |
| Type: | Improvement | Priority: | Minor - P4 |
| Reporter: | Richard Kreuter (Inactive) | Assignee: | Scott Hernandez (Inactive) |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||||||||||
| Participants: | |||||||||||||||||||||||||
| Description |
|
It seems like getParameter doesn't support retrieving all the tunables that setParameter does. That's just silly.
Here's the output under 2.2.0:
|
| Comments |
| Comment by auto [ 14/Aug/13 ] | ||||||||||||||||||||||||||||||||||
|
Author: {u'username': u'scotthernandez', u'name': u'Scott Hernandez', u'email': u'scotthernandez@gmail.com'}Message: | ||||||||||||||||||||||||||||||||||
| Comment by Bryan Reinero [ 28/Jan/13 ] | ||||||||||||||||||||||||||||||||||
|
Version 2.3.2 doesn't support getParameter on journalCommitInterval, although you can still set it. It's also absent in the commandHelp output: PRIMARY> db.adminCommand( { setParameter:1, journalCommitInterval:50 }) { "ok" : 1 }PRIMARY> db.adminCommand( { getParameter:1, journalCommitInterval:1 }) { "ok" : 0, "errmsg" : "no option found to get" }PRIMARY> db.commandHelp('getParameter') supported: to get everything PRIMARY> db.commandHelp('setParameter') supported: | ||||||||||||||||||||||||||||||||||
| Comment by Scott Hernandez (Inactive) [ 05/Jan/13 ] | ||||||||||||||||||||||||||||||||||
|
Fixed via other changes in set/getParameters in 2.3.2.
| ||||||||||||||||||||||||||||||||||
| Comment by auto [ 10/Sep/12 ] | ||||||||||||||||||||||||||||||||||
|
Author: {u'date': u'2012-09-09T13:44:47-07:00', u'email': u'richard@10gen.com', u'name': u'Richard Kreuter'}Message: Tad Marshall claims traceExceptions is plural. See | ||||||||||||||||||||||||||||||||||
| Comment by Richard Kreuter (Inactive) [ 09/Sep/12 ] | ||||||||||||||||||||||||||||||||||
|
Thanks, Tad. (traceExceptions with an 's' is apparently doc'd wrong, then, too. Pull request in progress.) | ||||||||||||||||||||||||||||||||||
| Comment by Tad Marshall [ 09/Sep/12 ] | ||||||||||||||||||||||||||||||||||
|
traceException should be traceExceptions, but it is still missing from getParameter. Also missing are ageOutJournalFiles, replMonitorMaxFailedChecks.
Help is incomplete for both getParameter and setParameter.
Ideally, the code should use a table for these parameters so that get, set and help are more easily kept in sync. | ||||||||||||||||||||||||||||||||||
| Comment by Richard Kreuter (Inactive) [ 09/Sep/12 ] | ||||||||||||||||||||||||||||||||||
|
Linking to the issue where setParameter and getParameter were created, for posterity's sake. | ||||||||||||||||||||||||||||||||||
| Comment by Richard Kreuter (Inactive) [ 09/Sep/12 ] | ||||||||||||||||||||||||||||||||||
|
If this improvement is made, the doc that needs altering is here: http://docs.mongodb.org/manual/reference/commands/#getParameter |