KEMBAR78
feat: provide sample code for row-level error handling by agrawal-siddharth · Pull Request #1863 · googleapis/java-bigquerystorage · GitHub
Skip to content

Conversation

@agrawal-siddharth
Copy link
Contributor

No description provided.

@agrawal-siddharth agrawal-siddharth requested a review from a team as a code owner October 31, 2022 18:36
@agrawal-siddharth agrawal-siddharth requested review from a team and shollyman October 31, 2022 18:36
@product-auto-label product-auto-label bot added the size: m Pull request size is medium. label Oct 31, 2022
@generated-files-bot
Copy link

Warning: This pull request is touching the following templated files:

  • samples/snippets/pom.xml

@product-auto-label product-auto-label bot added api: bigquerystorage Issues related to the googleapis/java-bigquerystorage API. samples Issues that are directly related to samples. labels Oct 31, 2022
@yirutang yirutang requested a review from mhlgio October 31, 2022 20:33
// Retry the remaining valid rows.
if (dataNew.length() > 0) {
try {
this.parent.append(new AppendContext(dataNew, 0));
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there a way to start another thread and do this append? Currently this append can block waiting for inflight limit...

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done. Both retries and row-errors are now handled by calls to append() made from a separate thread.

@shollyman shollyman requested review from Neenu1995 and removed request for shollyman November 6, 2022 19:08
@yirutang yirutang added the owlbot:run Add this label to trigger the Owlbot post processor. label Nov 14, 2022
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Nov 14, 2022
@yirutang yirutang merged commit 027c5bb into googleapis:main Nov 14, 2022
@agrawal-siddharth agrawal-siddharth deleted the rowErrorSampleCode branch November 18, 2022 00:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: bigquerystorage Issues related to the googleapis/java-bigquerystorage API. samples Issues that are directly related to samples. size: m Pull request size is medium.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants