-
Notifications
You must be signed in to change notification settings - Fork 6
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
DOCSP-42607: v5.2 changes #48
Conversation
✅ Deploy Preview for docs-kotlin-sync ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
LGTM with some small comments
to the expected types in BSON. | ||
|
||
In the following example, the driver serializes the fields 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.
S: [nit] Since this sentence repeats "following", you could reword to just "this example":
In the following example, the driver serializes the fields of | |
In this example, the driver serializes the fields of |
|
||
The following example inserts an instance of the ``Appointment`` data | ||
class into MongoDB: |
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.
S: you can be more specific about where you're inserting data
class into MongoDB: | |
class into a MongoDB collection named "appointments": |
|
||
In MongoDB, the ``LocalDate`` value is stored as a BSON date, and the | ||
``time`` field is stored as a string by default serialization: |
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.
S: remove extra space
``time`` field is stored as a string by default serialization: | |
``time`` field is stored as a string by default serialization: |
@@ -66,7 +72,8 @@ The following code example shows how to create an Atlas Search index: | |||
:end-before: end-create-search-index | |||
:dedent: | |||
|
|||
The following code example shows how to create multiple indexes: | |||
The following code example shows how to create Search and |
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.
S:
The following code example shows how to create Search and | |
The following code example shows how to create Atlas Search and |
pre-filter your data. | ||
|
||
You can call the following methods on a collection to manage your Atlas | ||
Search and Vector Search indexes: | ||
|
||
- ``createSearchIndex()`` | ||
- ``createSearchIndexes()`` |
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.
Q: Does the note below (can't comment on it directly) apply just to Atlas Search indexes? If it applies to Vector search index methods as well, I'd update it to reflect that
Pull Request Info
PR Reviewing Guidelines
JIRA - https://jira.mongodb.org/browse/DOCSP-42607
Staging:
Self-Review Checklist