[go: up one dir, main page]

Skip to content
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

fix: update TableInsertRows.java #2999

Merged
merged 2 commits into from Nov 28, 2023
Merged

Conversation

baeminbo
Copy link
Contributor
@baeminbo baeminbo commented Nov 13, 2023

Make the example set row id in addRow. If row id is missed, it disable the retry b/280865468, which I believe an unexpected behavior to users.

Fixes #3000.

@product-auto-label product-auto-label bot added size: s Pull request size is small. api: bigquery Issues related to the googleapis/java-bigquery API. labels Nov 13, 2023
@baeminbo baeminbo changed the title Update TableInsertRows.java fix: update TableInsertRows.java Nov 13, 2023
Copy link
conventional-commit-lint-gcf bot commented Nov 13, 2023

🤖 I detect that the PR title and the commit message differ and there's only one commit. To use the PR title for the commit history, you can use Github's automerge feature with squashing, or use automerge label. Good luck human!

-- conventional-commit-lint bot
https://conventionalcommits.org/

@baeminbo baeminbo force-pushed the patch-1 branch 2 times, most recently from f94d5be to 224e805 Compare November 13, 2023 04:45
@Neenu1995 Neenu1995 added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Nov 20, 2023
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Nov 20, 2023
@Neenu1995
Copy link
Contributor

@baeminbo Could you also fix the sample test to match the changes in code?

Make the example set row id in `addRow`. If row id is missed, it disable the retry b/280865468, which I believe an unexpected behavior to users.
@baeminbo
Copy link
Contributor Author

Hi, I updated TableInsertRowsIT. Now, I'm waiting on the integration test result.

@Neenu1995 Neenu1995 added kokoro:force-run Add this label to force Kokoro to re-run the tests. owlbot:run Add this label to trigger the Owlbot post processor. labels Nov 27, 2023
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Nov 27, 2023
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Nov 27, 2023
@baeminbo baeminbo requested review from a team as code owners November 27, 2023 21:54
@Neenu1995 Neenu1995 added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Nov 28, 2023
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Nov 28, 2023
@Neenu1995 Neenu1995 merged commit ff4a086 into googleapis:main Nov 28, 2023
16 checks passed
PhongChuong pushed a commit that referenced this pull request Dec 5, 2023
* fix: update TableInsertRows.java

Make the example set row id in `addRow`. If row id is missed, it disable the retry b/280865468, which I believe an unexpected behavior to users.

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

---------

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: bigquery Issues related to the googleapis/java-bigquery API. size: s Pull request size is small.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use row id for addRows in the sample TableInsertRows.java
3 participants