Skip to content

Do not merge: Validate gbek against a bunch of DF tests#36493

Closed
damccorm wants to merge 31 commits intomasterfrom
users/damccorm/validateGbek
Closed

Do not merge: Validate gbek against a bunch of DF tests#36493
damccorm wants to merge 31 commits intomasterfrom
users/damccorm/validateGbek

Conversation

@damccorm
Copy link
Copy Markdown
Contributor

Please add a meaningful description for your change here


Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

  • Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
  • Update CHANGES.md with noteworthy changes.
  • If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md

GitHub Actions Tests Status (on master branch)

Build python source distribution and wheels
Python tests
Java tests
Go tests

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

@damccorm damccorm changed the title Users/damccorm/validate gbek Do not merge: Validate gbek against a bunch of DF tests Oct 13, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Oct 13, 2025

Test Results

15 tests   - 2 187   15 ✅  - 1 966   1h 50m 50s ⏱️ - 1h 26m 6s
12 suites  -   209    0 💤  -   221 
12 files    -   209    0 ❌ ±    0 

Results for commit 1479726. ± Comparison against base commit 1e7167b.

This pull request removes 2204 and adds 15 tests. Note that renamed tests count towards both.
]
nanos: 534000000
org.apache.beam.sdk.io.gcp.GcpApiSurfaceTest ‑ testGcpApiSurface
org.apache.beam.sdk.io.gcp.bigquery.AvroGenericRecordToStorageApiProtoTest ‑ testCdcFields
org.apache.beam.sdk.io.gcp.bigquery.AvroGenericRecordToStorageApiProtoTest ‑ testDescriptorFromSchema
org.apache.beam.sdk.io.gcp.bigquery.AvroGenericRecordToStorageApiProtoTest ‑ testDescriptorFromSchemaLogicalTypes
org.apache.beam.sdk.io.gcp.bigquery.AvroGenericRecordToStorageApiProtoTest ‑ testMessageFromGenericRecord
org.apache.beam.sdk.io.gcp.bigquery.AvroGenericRecordToStorageApiProtoTest ‑ testMessageFromGenericRecordJavaTimeLogicalTypes
org.apache.beam.sdk.io.gcp.bigquery.AvroGenericRecordToStorageApiProtoTest ‑ testMessageFromGenericRecordJodaTimeLogicalTypes
org.apache.beam.sdk.io.gcp.bigquery.AvroGenericRecordToStorageApiProtoTest ‑ testMessageFromGenericRecordRawLogicalTypes
…
org.apache.beam.examples.complete.AutoCompleteIT ‑ testE2EAutoComplete
org.apache.beam.examples.complete.TfIdfIT ‑ testE2ETfIdf
org.apache.beam.examples.complete.TopWikipediaSessionsIT ‑ testE2ETopWikiPages
org.apache.beam.examples.complete.TrafficMaxLaneFlowIT ‑ testE2ETrafficMaxLaneFlow
org.apache.beam.examples.complete.TrafficRoutesIT ‑ testE2ETrafficRoutes
org.apache.beam.examples.cookbook.BigQueryTornadoesIT ‑ testE2EBigQueryTornadoesWithExport
org.apache.beam.examples.cookbook.BigQueryTornadoesIT ‑ testE2EBigQueryTornadoesWithExportUsingQuery
org.apache.beam.examples.cookbook.BigQueryTornadoesIT ‑ testE2eBigQueryTornadoesWithStorageApi
org.apache.beam.examples.cookbook.BigQueryTornadoesIT ‑ testE2eBigQueryTornadoesWithStorageApiUsingQuery
org.apache.beam.examples.cookbook.CombinePerKeyExamplesIT ‑ testE2ECombinePerKey
…

♻️ This comment has been updated with latest results.

@codecov
Copy link
Copy Markdown

codecov Bot commented Oct 14, 2025

Codecov Report

❌ Patch coverage is 12.50000% with 14 lines in your changes missing coverage. Please review.
✅ Project coverage is 36.21%. Comparing base (e3293e2) to head (1479726).
⚠️ Report is 67 commits behind head on master.

Files with missing lines Patch % Lines
...dks/python/apache_beam/options/pipeline_options.py 22.22% 7 Missing ⚠️
sdks/python/apache_beam/transforms/util.py 0.00% 6 Missing ⚠️
...ache_beam/runners/portability/expansion_service.py 0.00% 1 Missing ⚠️
Additional details and impacted files
@@              Coverage Diff              @@
##             master   #36493       +/-   ##
=============================================
- Coverage     55.03%   36.21%   -18.82%     
  Complexity     1667     1667               
=============================================
  Files          1058     1058               
  Lines        165248   165295       +47     
  Branches       1190     1190               
=============================================
- Hits          90940    59866    -31074     
- Misses        72134   103255    +31121     
  Partials       2174     2174               
Flag Coverage Δ
python 40.53% <12.50%> (-40.42%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 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.

@damccorm damccorm closed this Oct 16, 2025
@damccorm damccorm deleted the users/damccorm/validateGbek branch October 16, 2025 13:34
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