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
chore: java doc update to for returned data instances #165
Conversation
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.
Can we also add an issue for this in gax to update the Batcher interface?
google-cloud-bigtable/src/main/java/com/google/cloud/bigtable/data/v2/BigtableDataClient.java
Outdated
Show resolved
Hide resolved
google-cloud-bigtable/src/main/java/com/google/cloud/bigtable/data/v2/BigtableDataClient.java
Outdated
Show resolved
Hide resolved
also - conventional commits check is failing I believe because there is not a space between |
To let the user know about Batcher instance returned from Bulk operations are not threadsafe.
Codecov Report
@@ Coverage Diff @@
## master #165 +/- ##
=========================================
Coverage ? 81.92%
Complexity ? 971
=========================================
Files ? 99
Lines ? 6021
Branches ? 330
=========================================
Hits ? 4933
Misses ? 910
Partials ? 178
Continue to review full report at Codecov.
|
Added JavaDocs for `BigtableDataClient`.
Thanks for the review, I have addressed previous comments, Please have a fresh look. |
google-cloud-bigtable/src/main/java/com/google/cloud/bigtable/data/v2/BigtableDataClient.java
Show resolved
Hide resolved
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.
Looks good, but readRows needs a note as well
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
Fixes #114