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.

Sign up for GitHub

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

Jump to bottom

bigquery/storage/managedwriter: deadlocks due to locking changes #6278

Closed
shollyman opened this issue Jun 29, 2022 · 0 comments · Fixed by #6304
Closed

bigquery/storage/managedwriter: deadlocks due to locking changes #6278

shollyman opened this issue Jun 29, 2022 · 0 comments · Fixed by #6304
Assignees
@shollyman
Labels
api: bigquery Issues related to the BigQuery API. priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.

Comments

@shollyman
Copy link
Contributor

#6243 changed the surface area of the critical sections, and while addressing one problem introduced new unlocking problems, particularly for error returns.

Need to address this.
@shollyman shollyman added type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns. api: bigquery Issues related to the BigQuery API. priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. labels Jun 29, 2022
@shollyman shollyman self-assigned this Jun 29, 2022
This was referenced Jun 29, 2022
Merged
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@shollyman shollyman

Labels
api: bigquery Issues related to the BigQuery API. priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.
Projects
None yet
Milestone
No milestone
1 participant
@shollyman