diff options
| author | yomegui <garnaud25@gmail.com> | 2015-12-23 17:24:45 +0100 |
|---|---|---|
| committer | Mike O'Brien <mpobrien005@gmail.com> | 2016-01-05 14:16:39 -0500 |
| commit | 90d1e5ee9a7b5187dace99aafe196252c0057c29 (patch) | |
| tree | 3d0b9f0ab6f55a355ac03c698941292775dd8815 | |
| parent | 92165bfbaefba2b6cad3a2546c56b376a44fc715 (diff) | |
| download | mongo-90d1e5ee9a7b5187dace99aafe196252c0057c29.tar.gz | |
mongostat json has wrong mmapv1-specific fields
The json formatter doesn't display these fields when the engine is WiredTiger:
* flushes
* ar|aw
* qr|qw
But these fields are present in grid formatter and documentation doesn't mention that these fields are mmapv1 specific (https://docs.mongodb.org/manual/reference/program/mongostat/).
This patch fixes that.
| -rw-r--r-- | mongostat/stat_types.go | 11 |
1 files changed, 4 insertions, 7 deletions
diff --git a/mongostat/stat_types.go b/mongostat/stat_types.go index 6602f795a1c..492d162a618 100644 --- a/mongostat/stat_types.go +++ b/mongostat/stat_types.go @@ -461,15 +461,12 @@ func (jlf *JSONLineFormatter) FormatLines(lines []StatLine, index int, discover lineJson["host"] = line.Host lineJson["vsize"] = text.FormatMegabyteAmount(int64(line.Virtual)) lineJson["res"] = text.FormatMegabyteAmount(int64(line.Resident)) - + lineJson["flushes"] = fmt.Sprintf("%v", line.Flushes) + lineJson["qr|qw"] = fmt.Sprintf("%v|%v", line.QueuedReaders, line.QueuedWriters) + lineJson["ar|aw"] = fmt.Sprintf("%v|%v", line.ActiveReaders, line.ActiveWriters) + // add mmapv1-specific fields if lineFlags&MMAPOnly > 0 { - lineJson["flushes"] = fmt.Sprintf("%v", line.Flushes) - lineJson["qr|qw"] = fmt.Sprintf("%v|%v", line.QueuedReaders, - line.QueuedWriters) - lineJson["ar|aw"] = fmt.Sprintf("%v|%v", line.ActiveReaders, - line.ActiveWriters) - mappedVal := "" // empty for mongos if line.Mapped > 0 { // not mongos, update accordingly mappedVal = text.FormatMegabyteAmount(int64(line.Mapped)) |