Skip to content

Conversation

mcmorisi
Copy link
Collaborator

Pull Request Info

PR Reviewing Guidelines

JIRA - https://jira.mongodb.org/browse/DOCSP-41142
Staging -

Self-Review Checklist

  • Is this free of any warnings or errors in the RST?
  • Did you run a spell-check?
  • Did you run a grammar-check?
  • Are all the links working?
  • Are the facets and meta keywords accurate?

Copy link

netlify bot commented Jul 19, 2024

👷 Deploy request for docs-kotlin-sync pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 01aec73

Copy link
Collaborator

@shuangela shuangela left a comment

Choose a reason for hiding this comment

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

nice work, some comments/questions for you.

your application's queries and operations that return sorted results. Each
index that you add consumes disk space and memory when active, so we recommend
that you track index memory and disk usage for capacity planning. In addition,
when a write operation updates an indexed field, MongoDB updates the related
Copy link
Collaborator

Choose a reason for hiding this comment

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

S: I see that in the server manual, it talks about how indexes are not always the best when using write operations. I believe that the sentence starting with "In addition, when a write.." is implying that indices can negatively affect write operation performance, but it would be great to make that clearer.

index.

Because MongoDB supports dynamic schemas, applications can query against fields
whose names are not known in advance or are arbitrary. MongoDB 4.2 introduced
Copy link
Collaborator

Choose a reason for hiding this comment

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

Q: Is it it necessary to say what version of MongoDB introduced wildcard indexes, or can we just say that wildcard indexes support queries with fields that have arbitrary/unknown names?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Can reword.


.. note::

You cannot remove a single field from a compound text index. You must
Copy link
Collaborator

Choose a reason for hiding this comment

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

Q: I don't know what a compound text index is; is that information you expect the user to have already going into this page?

Copy link
Collaborator Author

@mcmorisi mcmorisi Jul 19, 2024

Choose a reason for hiding this comment

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

All types of indexes are slated to receive their own page – this page will link to the compound text index page once it's written.

Create Single-Field Index
-------------------------

The following example creates an index in ascending order on the ``title`` field:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Q:Are there options to create an index in any other order other than ascending and descending?

S: Clarify/state explicitly that you can create an index in either ascending or descending order.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Will add to page overview.

@mcmorisi mcmorisi requested a review from shuangela July 22, 2024 13:10
Copy link
Collaborator

@shuangela shuangela left a comment

Choose a reason for hiding this comment

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

lgtm 💯

@mcmorisi mcmorisi merged commit 12ef143 into mongodb:master Jul 22, 2024
0 of 2 checks passed
@mcmorisi mcmorisi deleted the DOCSP-41142-single-field-indexes branch July 22, 2024 13:49
mcmorisi added a commit that referenced this pull request Jul 22, 2024
mcmorisi added a commit that referenced this pull request Jul 22, 2024
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.

2 participants