-
Notifications
You must be signed in to change notification settings - Fork 24
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-47834 clarify batch size #524
DOCSP-47834 clarify batch size #524
Conversation
✅ Deploy Preview for docs-csharp 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!
batch size of ``101`` documents and a maximum size of 16 | ||
mebibytes (MiB) for each subsequent batch. This option can | ||
enforce a smaller limit than 16 MiB, but not a larger one. If | ||
you set ``batchSize`` to a limit that results in batches |
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 set ``batchSize`` to a limit that results in batches | |
you set ``batchSize`` to a limit that results in batches |
* - ``batchSize`` | ||
- | Specifies the maximum number of documents that a change | ||
stream can return in each batch, which applies to ``Watch()`` or | ||
``WatchAsync()``. By default, watch functions have an initial |
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.
We should clarify what "By default" means: that batchSize
is optional and if batchSize
parameter is not set watch function have...
Otherwise it might sound like the default is batchSize == 101 in the option objects.
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 lead in text to the table states that all these options are optional, and I added a clarification on what "by default" means on both this page and the other page I edited!
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!
* clarify batch size * change find * clarify no effect * fix wrapping * remove extra space * review (cherry picked from commit 9d38c02)
* clarify batch size * change find * clarify no effect * fix wrapping * remove extra space * review (cherry picked from commit 9d38c02)
* clarify batch size * change find * clarify no effect * fix wrapping * remove extra space * review (cherry picked from commit 9d38c02)
* clarify batch size * change find * clarify no effect * fix wrapping * remove extra space * review (cherry picked from commit 9d38c02)
* clarify batch size * change find * clarify no effect * fix wrapping * remove extra space * review (cherry picked from commit 9d38c02)
* clarify batch size * change find * clarify no effect * fix wrapping * remove extra space * review (cherry picked from commit 9d38c02)
* clarify batch size * change find * clarify no effect * fix wrapping * remove extra space * review (cherry picked from commit 9d38c02)
* clarify batch size * change find * clarify no effect * fix wrapping * remove extra space * review (cherry picked from commit 9d38c02)
* clarify batch size * change find * clarify no effect * fix wrapping * remove extra space * review (cherry picked from commit 9d38c02)
* clarify batch size * change find * clarify no effect * fix wrapping * remove extra space * review (cherry picked from commit 9d38c02) Co-authored-by: shuangela <[email protected]>
* clarify batch size * change find * clarify no effect * fix wrapping * remove extra space * review (cherry picked from commit 9d38c02) Co-authored-by: shuangela <[email protected]>
* clarify batch size * change find * clarify no effect * fix wrapping * remove extra space * review (cherry picked from commit 9d38c02) Co-authored-by: shuangela <[email protected]>
* clarify batch size * change find * clarify no effect * fix wrapping * remove extra space * review (cherry picked from commit 9d38c02) Co-authored-by: shuangela <[email protected]>
* clarify batch size * change find * clarify no effect * fix wrapping * remove extra space * review (cherry picked from commit 9d38c02) Co-authored-by: shuangela <[email protected]>
* clarify batch size * change find * clarify no effect * fix wrapping * remove extra space * review (cherry picked from commit 9d38c02) Co-authored-by: shuangela <[email protected]>
* clarify batch size * change find * clarify no effect * fix wrapping * remove extra space * review (cherry picked from commit 9d38c02) Co-authored-by: shuangela <[email protected]>
* clarify batch size * change find * clarify no effect * fix wrapping * remove extra space * review (cherry picked from commit 9d38c02) Co-authored-by: shuangela <[email protected]>
* clarify batch size * change find * clarify no effect * fix wrapping * remove extra space * review (cherry picked from commit 9d38c02) Co-authored-by: shuangela <[email protected]>
Pull Request Info
PR Reviewing Guidelines
JIRA - https://jira.mongodb.org/browse/DOCSP-47834
Staging Links
Self-Review Checklist