-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Conn pool stats #65
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
Conn pool stats #65
Conversation
output of the command. | ||
|
||
Detailed description of the output of | ||
:dbcommand:`connPoolStats`. For each MongoDB host connected: |
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.
You don't need to tell people that the description is detailed.
"MongoDB host" is not a term that appears in the Manual. Use :program:mongod
(I think) in these instances.
can you revise and update your branch. talk to me if you need help adding the directives. |
|
||
This field indicates the number of the host that is the next | ||
:term:`secondary` to read from instead of :term:`master` from | ||
the ``hosts`` list. |
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.
can you fix this and the previous entry.
.. stats:: hosts.[host].available | ||
|
||
The cummulative number of connections in the connection pool | ||
available for a host to connect to a :program:`mongos`. |
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.
Please rewrite this and all following sections to be:
- Less tautological. The current forms are vague, and do not add value to the documentation.
- Less passive.
- Technically correct. The connection pool is something that the mongs has to connect to the members of the replica set. These don't have that factual point correct.
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.
for example:
:stats:`hosts.[host].available` reports the total number of connections created by the :program:`mongos`, and used to connect to members of the replica set.
Please revise and resubmit. |
.. note:: | ||
|
||
:dbcommand:`connPoolStats` only return meaningful results for | ||
:program:`mongos` program. |
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.
I don't think this is true. connPoolStats can be useful on a mongod as well.
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.
can you elaborate? on this. will try to touch base with you in person to resolve this.
|
||
:stats:`hosts.[host].available` reports the total number of | ||
connections created by the :program:`mongos`, and used to | ||
connect to members of the :term:`shard cluster`. |
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.
The available and created counters are per host, so it's not the number of available conns for the whole shard cluster, it's the number available to connect to a specific shard.
* DOCSP-19786 validate warning clarification * Fix whitespace * Update tip * White space * Review updates * Remove tip * External feedback * Tweaks * Move primary note to warning
Co-authored-by: jason-price-mongodb <[email protected]>
* DOCSP-33454 Query Converter Content (#58) * DOCSP-33454 Query Converter Content * DOCSP-32086 Convert Stored Procedures Content (#64) * DOCSP-32086 Convert Stored Procedures Content * DOCSP-33246 Convert Views Content (#63) * DOCSP-33246 Convert Views Content * DOCSP-33003 Refresh Database Objects (#65) * remove empty file
* Adds release notes for 1.1.6 and 1.1.7, updates source constant * Sets submodule to atlascli/v1.1.7 * Includes change from Julia's copy review
* (DOCSP-11398): Update Telephone Numbers page * copy review
…ce the wrong year (#65) (#68) - [JIRA](https://jira.mongodb.org/browse/DOCSP-36081) - [Stage](https://preview-mongodbkanchanamongodb.gatsbyjs.io/ops-manager/DOCSP-36081/release-notes/application/) - [Build](https://workerpool-boxgs.mongodbstitch.com/pages/job.html?collName=queue&jobId=65b954056b2f17c95af6303f)
…ce the wrong year (#65) (#69) - [JIRA](https://jira.mongodb.org/browse/DOCSP-36082) - [Stage](https://preview-mongodbkanchanamongodb.gatsbyjs.io/ops-manager/DOCSP-36082/release-notes/application/) - [Build](https://workerpool-boxgs.mongodbstitch.com/pages/job.html?collName=queue&jobId=65b9548a6b2f17c95af67280)
DOCS-158
Please review, update, and merge.
Thanks.