-
Notifications
You must be signed in to change notification settings - Fork 1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add "couch_server" aggregate to _system output #3370
Conversation
6ba642d
to
9eaa75d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Couple nits.
src/couch/src/couch_server.erl
Outdated
@@ -937,6 +938,14 @@ num_servers() -> | |||
erlang:system_info(schedulers). | |||
|
|||
|
|||
aggregate_queue_len() -> | |||
N = couch_server:num_servers(), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are you using the fully qualified external call on purpose for reasons I'm not thinking of?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
erm, nope. In the first commit it was in chttpd_node
src/chttpd/src/chttpd_node.erl
Outdated
@@ -289,6 +290,7 @@ message_queues(Registered) -> | |||
{Name, Length} | |||
end, Registered). | |||
|
|||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
White space only hunks are not awesome.
9eaa75d
to
379bf55
Compare
379bf55
to
21bc72c
Compare
This helps ease transition from singleton couch_server to multiple. The "couch_server" message queue is simply the sum of the couch_server_X message queues.
21bc72c
to
5724e8e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1
Overview
This helps ease transition from singleton couch_server to
multiple. The "couch_server" message queue is simply the sum of the
couch_server_X message queues.
Testing recommendations
N/A
Related Issues or Pull Requests
#3366
Checklist
rel/overlay/etc/default.ini