Skip to content

fix for globalLock.ration value - #8

Open
michalekn wants to merge 2 commits into
erh:masterfrom
michalekn:master
Open

fix for globalLock.ration value#8
michalekn wants to merge 2 commits into
erh:masterfrom
michalekn:master

Conversation

@michalekn

Copy link
Copy Markdown

Since mongo 2.2 there is no info in globalLock.ratio but it can be counted manually by dividing globalLock.lockTime and globalLock.totalTime. This fix doing it.

@tianon

tianon commented Nov 6, 2012

Copy link
Copy Markdown

It would also be useful if it were to loop over the value of getServerStatus()["locks"] and give us a ratio for each one, since MongoDB now does database-level locking, and these values represent the lock time for each database separately.

@jlecour

jlecour commented Nov 8, 2012

Copy link
Copy Markdown

@tianon It's a detail, but he correct command is db.serverStatus()["locks"].

Otherwise, I get this :

> db.getServerStatus()["locks"]
Thu Nov  8 17:38:10 TypeError: db.getServerStatus is not a function (shell):1

@jlecour

jlecour commented Nov 8, 2012

Copy link
Copy Markdown

My bad, I didn't see that the function was from the script, not from MongoDB itself.

Sorry for the n00b noise.

@tianon

tianon commented Nov 8, 2012

Copy link
Copy Markdown

No worries; I had the exact same thought initially until I looked closer at
the script.

  • Andrew

On 8 November 2012 10:05, Jérémy Lecour notifications@github.com wrote:

My bad, I didn't see that the function was from the script, not from
MongoDB itself.

Sorry for the n00b noise.


Reply to this email directly or view it on GitHubhttps://github.com//pull/8#issuecomment-10195901.

@jlecour

jlecour commented Nov 8, 2012

Copy link
Copy Markdown

I've tried this patch, and I get a growing value, always > 1.0 which doesn't look like a percentage.

RBLU pushed a commit to YouPers/mongo-munin that referenced this pull request Feb 16, 2016
fixed how to configure custom db connection
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants