Skip to content

Conversation

@ravindk89
Copy link
Contributor

@ravindk89 ravindk89 commented Dec 16, 2017

@ravindk89 ravindk89 requested a review from kay-kim December 16, 2017 02:23
Copy link
Contributor

@shannonbradshaw shannonbradshaw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Two small items to address. Otherwise, LGTM

switch to the ``local`` database:
The following operation displays the oplog size using the
:method:`db.collection.stats()` command and
:data:`~collStats.maxSize`:
Copy link
Contributor

@shannonbradshaw shannonbradshaw Dec 19, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This paragraph is unnecessary. Please remove.

:method:`db.collection.stats()` command and
:data:`~collStats.maxSize`:

.. code-block:: javascript
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This code block should be copyable. The other codeblocks are fine as is.

@ravindk89
Copy link
Contributor Author

@shannonbradshaw done, restaged. @kay-kim , I need to squash before merge (or we can merge as is).

@kay-kim kay-kim merged commit a3a81e6 into mongodb:master Dec 22, 2017
@ravindk89 ravindk89 deleted the DOCS-11134 branch April 11, 2018 22:13
mongo-cr-bot pushed a commit that referenced this pull request May 23, 2023
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