KEMBAR78
feat: increase timeout waiting for done callback from 2 minutes to 5 … by yirutang · Pull Request #1682 · googleapis/java-bigquerystorage · GitHub
Skip to content

Conversation

@yirutang
Copy link
Contributor

…minutes

In my stress test I am seeing a lot of CANCELLED: Timeout waiting for DoneCallback. failure when I am making calls to EU region, which led me to think the timeout is too short.

@yirutang yirutang requested review from a team and prash-mi June 22, 2022 20:54
@product-auto-label product-auto-label bot added size: xs Pull request size is extra small. api: bigquerystorage Issues related to the googleapis/java-bigquerystorage API. labels Jun 22, 2022
@prash-mi
Copy link
Contributor

@yirutang Thanks for raising the PR. Quick question - where did you run the code from (local vs a VM?), also did the issue resolve with 5 minutes as the timeout?

@alicejli
Copy link
Contributor

alicejli commented Jul 12, 2022

@yirutang gentle bump. Could you elaborate on your findings from your stress test? Thanks!

I am running a stress test run that writes as much data as possible from a VM and the test saw a lot of such errors.

@Neenu1995 Neenu1995 added the owlbot:run Add this label to trigger the Owlbot post processor. label Jul 26, 2022
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Jul 26, 2022
@yirutang yirutang requested a review from a team as a code owner July 26, 2022 21:08
@yirutang yirutang merged commit 5171898 into googleapis:main Jul 26, 2022
@yirutang
Copy link
Contributor Author

yirutang commented Jul 26, 2022

@yirutang Thanks for raising the PR. Quick question - where did you run the code from (local vs a VM?), also did the issue resolve with 5 minutes as the timeout?

I ran it from a VM. Running it from local doesn't seem to have an issue.

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. size: xs Pull request size is extra small.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants