-
Notifications
You must be signed in to change notification settings - Fork 89
fix: return row-level error information via an AppendSerializtionError #1813
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
Conversation
| LOG.info("Sending three messages"); | ||
| ApiFuture<AppendRowsResponse> futureResponse = | ||
| streamWriter.append( | ||
| CreateProtoRows(new String[] {"aaabbbcccddd", "bbb", "cccdddeeefffggg"}), -1); |
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.
Shouldn't block this checkin, could you try to hit the timestamp error in the bug to verify that path is fixed?
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.
Some of the presubmit checks are failing, such as the title of the PR needs to be:
fix: xxxx
f85dd62 to
0590379
Compare
* chore: Update `dependabot.yml` template not to touch pip dependencies Source-Link: googleapis/synthtool@f961eb0 Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-java:latest@sha256:af2eda87a54601ae7b7b2be5055c17b43ac98a7805b586772db314de8a7d4a1d
…#1199) * chore: Update `dependabot.yml` template not to touch pip dependencies Source-Link: googleapis/synthtool@f961eb0 Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-java:latest@sha256:af2eda87a54601ae7b7b2be5055c17b43ac98a7805b586772db314de8a7d4a1d Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
BEGIN_COMMIT_OVERRIDE
feat: return list of row errors from append
fix: return row-level error information via an AppendSerializtionError exception
END_COMMIT_OVERRIDE