Skip to content

test(bigquerystorage): run flaky public-table read tests on v1 only - #14991

Merged
lqiu96 merged 1 commit into
googleapis:mainfrom
lqiu96:test/bqstorage-v1-only-reads
Nov 12, 2026
Merged

lqiu96 merged 1 commit into
googleapis:mainfrom
lqiu96:test/bqstorage-v1-only-reads

Conversation

@lqiu96

@lqiu96 lqiu96 commented Nov 12, 2026

Copy link
Copy Markdown
Member

Re-enable testSimpleReadAndResume and testUniverseDomainWithMatchingDomain in v1.it.ITBigQueryStorageReadClientTest while disabling testSimpleReadAndResume in v1beta1 and v1beta2 ITBigQueryStorageTest so duplicate public-dataset CreateReadSession tests run on v1 only.

The assumption is that these test failures are due to the high load for the underlying CreateReadSession calls. Running this only for v1 and not v1beta1 and v1beta2 should help with the occasional INTERNAL errors. v1 is the public stable API and v1beta1 and v1beta2 are older version that are not actively developed on.

From b/564596647

Re-enable testSimpleReadAndResume and testUniverseDomainWithMatchingDomain in v1.it.ITBigQueryStorageReadClientTest while disabling testSimpleReadAndResume in v1beta1 and v1beta2 ITBigQueryStorageTest so duplicate public-dataset CreateReadSession tests run on v1 only.

Refs: b/564596647
@lqiu96
lqiu96 requested a review from jinseopkim0 November 12, 2026 17:30
@lqiu96
lqiu96 requested review from a team as code owners November 12, 2026 17:30

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request re-enables two integration tests (testSimpleReadAndResume and testUniverseDomainWithMatchingDomain) in ITBigQueryStorageReadClientTest.java by removing their @Disabled annotations. Conversely, it disables the flaky testSimpleReadAndResume test in both v1beta1 and v1beta2 versions of ITBigQueryStorageTest.java by adding the @Disabled annotation. There are no review comments to evaluate, and I have no additional feedback to provide.

@lqiu96
lqiu96 enabled auto-merge (squash) November 12, 2026 17:35
@lqiu96
lqiu96 merged commit a117804 into googleapis:main Nov 12, 2026
205 of 206 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants