Skip to content

Fix DataStream test timeout - #4159

Draft
Abacn wants to merge 4 commits into
GoogleCloudPlatform:mainfrom
Abacn:fix-datastream-test
Draft

Fix DataStream test timeout#4159
Abacn wants to merge 4 commits into
GoogleCloudPlatform:mainfrom
Abacn:fix-datastream-test

Conversation

@Abacn

@Abacn Abacn commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Datastream finalizes and rotates GCS files every 60 seconds, triggering GCS OBJECT_FINALIZE Pub/Sub notifications promptly so the integration test receives data and completes normally without hitting the condition check or test timeouts.

Datastream finalizes and rotates GCS files every 60 seconds, triggering GCS OBJECT_FINALIZE Pub/Sub notifications promptly so the integration test receives data and completes normally without hitting the condition check or test timeouts.
@codecov

codecov Bot commented Aug 18, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 94.28571% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 55.89%. Comparing base (9d8862d) to head (db7dd03).
⚠️ Report is 33 commits behind head on main.

Files with missing lines Patch % Lines
...m/it/gcp/datastream/DatastreamResourceManager.java 94.28% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main    #4159      +/-   ##
============================================
- Coverage     56.21%   55.89%   -0.33%     
- Complexity     7340     7475     +135     
============================================
  Files          1126     1134       +8     
  Lines         68748    70136    +1388     
  Branches       7784     8011     +227     
============================================
+ Hits          38650    39205     +555     
- Misses        27599    28386     +787     
- Partials       2499     2545      +46     
Components Coverage Δ
spanner-templates 84.63% <ø> (-3.00%) ⬇️
spanner-import-export 68.93% <ø> (-0.11%) ⬇️
spanner-live-forward-migration 88.66% <ø> (-0.92%) ⬇️
spanner-live-reverse-replication 81.26% <ø> (-2.45%) ⬇️
spanner-bulk-migration 89.06% <ø> (-3.24%) ⬇️
gcs-spanner-dv 87.85% <ø> (-1.06%) ⬇️
Files with missing lines Coverage Δ
...m/it/gcp/datastream/DatastreamResourceManager.java 78.79% <94.28%> (+0.69%) ⬆️

... and 52 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@pull-request-size pull-request-size Bot added size/M and removed size/S labels Aug 21, 2026
@pull-request-size pull-request-size Bot added size/L and removed size/M labels Aug 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant