[SERVER-19941] enable tcmalloc stats in serverStatus by default Created: 13/Aug/15  Updated: 28/Sep/16  Resolved: 11/Sep/15

Status: Closed
Project: Core Server
Component/s: Diagnostics
Affects Version/s: None
Fix Version/s: 3.1.8

Type: Improvement Priority: Major - P3
Reporter: Bruce Lucas (Inactive) Assignee: Mathias Stearn
Resolution: Done Votes: 1
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Documented
is documented by DOCS-8964 enable tcmalloc stats in serverStatus... Closed
Duplicate
is duplicated by SERVER-19931 Overflow in db.serverStatus().extra_info Closed
Backwards Compatibility: Minor Change
Sprint: Quint 9 09/18/15
Participants:

 Description   

Currently you must specify db.serverStatus({tcmalloc:true}) to get tcmalloc stats. It would be useful from a support perspective if this were enabled by default so that we have this information in any serverStatus data that is collected.



 Comments   
Comment by Githook User [ 11/Sep/15 ]

Author:

{u'username': u'RedBeard0531', u'name': u'Mathias Stearn', u'email': u'mathias@10gen.com'}

Message: SERVER-19941 enable tcmalloc ServerStatusSection by default
Branch: master
https://github.com/mongodb/mongo/commit/d03334dfa87386feef4b8331f0e183d80495808c

Comment by Ben McCann [ 01/Sep/15 ]

Thanks!

Comment by Mathias Stearn [ 01/Sep/15 ]

chengas123 that is the plan at the moment. There is no harm in explicitly requesting a section that defaults to on, if that is what you are asking. However, we haven't yet done any benchmarking to ensure that there is no performance hit, so it may be best not to do it at a high frequency on a production system until that is confirmed. We will do that benchmarking before 3.2 is released.

Comment by Ben McCann [ 01/Sep/15 ]

@Mathias does the change to "3.1 Required" mean you do plan to change it to be on by default? I'm curious because I have some monitoring I wrote which polls

db.serverStatus()

and I'm wondering if there'd be any harm in having it instead poll

db.serverStatus({tcmalloc:true})

Comment by Martin Bligh [ 25/Aug/15 ]

Assigning to redbeard0531 - he wrote this, per SERVER-6950 seems to have been deliberately off by default, but unspecified as to why, or if that's really necessary

Comment by Ben McCann [ 13/Aug/15 ]

+1 any reason this isn't included by default already?

Generated at Thu Feb 08 03:52:38 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.