Details
-
Improvement
-
Resolution: Fixed
-
Major - P3
-
None
-
None
-
None
-
Fully Compatible
-
Dev Tools 2018-12-31
Description
We use ServerStatusSection to populate the server status throughout the codebase, but it has come to our attention that we are doing it incorrectly. Instead of
virtual bool includeByDefault() const |
|
|
and
|
|
|
virtual BSONObj generateSection(OperationContext* opCtx, const BSONElement& configElement) const |
it should be:
bool includeByDefault() const override |
|
|
and
|
|
|
BSONObj generateSection(OperationContext* opCtx, const BSONElement& configElement) const override |
So just git grep "ServerStatusSections" to find the relevant areas and change them to avoid future bugs.