-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Improve the toCollection
and related API documentation
#1677
base: main
Are you sure you want to change the base?
Conversation
ff4cfe4
to
d73f992
Compare
…shouldHaveTheSameMethods` JAVA-5833
We have the same test failures here, where they are obviously not caused by the change. Therefore, the failures here are also not caused by the changes. |
Replaced Ross with @nhachicha, because Ross is away. |
/** | ||
* Requests [AggregateFlow] to start streaming data according to the specified aggregation pipeline. | ||
* - If the aggregation pipeline ends with an `$out` or `$merge` stage, then finds all documents in the affected | ||
* namespace and emits them. You may want to use [toCollection] instead. | ||
* - Otherwise, emits no values. | ||
*/ |
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.
Note to others that these docs and some others are new, and therefore non-trivial.
@rozza is the primary reviewer (chosen automatically), @katcharov is the secondary one (chosen manually based on the GitHub suggestion).
JAVA-5833