-
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
Add Kotlin Bulk Write API #1591
Conversation
- Add sync and coroutine Bulk Write API. - Enable unified spec tests. JAVA-5532
JAVA-5532
JAVA-5532
JAVA-5532
JAVA-5532
* Executes a client-level bulk write operation. This method is functionally equivalent to | ||
* [bulkWrite(models, options)][bulkWrite] with the | ||
* [default options][ClientBulkWriteOptions.clientBulkWriteOptions]. |
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.
Kotlin does not support referencing methods with parameters, as noted here: KT-15984 and in their Kotlin Documentation.
Note that KDoc does not have any syntax for resolving overloaded members in links. Since Kotlin's documentation generation tool puts the documentation for all overloads of a function on the same page, identifying a specific overloaded function is not required for the link to work.
Therefore, links in KDoc are written with additional descriptive text to clarify the context of the reference.
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
I did not check or diff the docs, since they were pasted in.
Primary review: @katcharov
Secondary (optional): @stIncMale
JAVA-5532