Skip to content

Add gcs_avro_to_bigtable yaml blueprint example and e2e test - #40484

Open
derrickaw wants to merge 4 commits into
apache:masterfrom
derrickaw:gcs_avro_to_cloud_bigtable
Open

Add gcs_avro_to_bigtable yaml blueprint example and e2e test#40484
derrickaw wants to merge 4 commits into
apache:masterfrom
derrickaw:gcs_avro_to_cloud_bigtable

Conversation

@derrickaw

@derrickaw derrickaw commented Oct 9, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator

@derrickaw derrickaw changed the title Add gcs_avro_to_cloud_bigtable yaml blueprint example and e2e test Add gcs_avro_to_bigtable yaml blueprint example and e2e test Oct 9, 2026
@codecov

codecov Bot commented Oct 9, 2026 •
edited
Loading

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.72%. Comparing base (76f6e6d) to head (4afa366).
⚠️ Report is 143 commits behind head on master.

Additional details and impacted files 
@@              Coverage Diff              @@
##             master   #40484       +/-   ##
=============================================
+ Coverage     56.16%   79.72%   +23.55%     
=============================================
  Files          1124      487      -637     
  Lines        178336    82280    -96056     
  Branches       1489        0     -1489     
=============================================
- Hits         100170    65601    -34569     
+ Misses        75645    16679    -58966     
+ Partials       2521        0     -2521
Flag Coverage Δ
go ?
java ?
python 79.72% <ø> (+0.17%) ⬆️

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

☔ View full report in Codecov by Harness.
📢 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.

@derrickaw
derrickaw marked this pull request as ready for review October 9, 2026 16:56
@github-actions

github-actions Bot commented Oct 9, 2026

Copy link
Copy Markdown
Contributor

Assigning reviewers:

R: @damccorm for label python.
R: @chamikaramj for label yaml.

Note: If you would like to opt out of this review, comment assign to next reviewer.

Available commands:

  • stop reviewer notifications - opt out of the automated review tooling
  • remind me after tests pass - tag the comment author after tests pass
  • waiting on author - shift the attention set back to the author (any comment or push by the author will return the attention set to the reviewers)

The PR bot will only process comments in the main thread (not review comments).

@derrickaw
derrickaw force-pushed the gcs_avro_to_cloud_bigtable branch from 839deb4 to 9bc2827 Compare October 12, 2026 17:39
damccorm
damccorm reviewed Oct 12, 2026
View reviewed changes
Comment thread sdks/python/apache_beam/yaml/examples/transforms/blueprint/gcs_avro_to_cloud_bigtable.yaml Outdated

# This is an example of a Beam YAML pipeline that reads from GCS Avro files
# and writes to Cloud Bigtable. This matches the Dataflow Template located
# here - https://cloud.google.com/dataflow/docs/guides/templates/provided/avro-to-cloud-bigtable

@damccorm damccorm Oct 12, 2026

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.

What is our goal in mirroring all of these across both places? Do we intend to keep them totally in sync?

@damccorm damccorm Oct 12, 2026

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.

It looks like we've been doing this for a while, but this seems like an antipattern to me

@damccorm damccorm Oct 12, 2026

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.

Or are we just saying that this has the same functionality as the Java version of this template?

Btw, is there a reason we're back to blueprint terminology? That may be what is confusing me (#39549 (comment))

@derrickaw derrickaw Oct 13, 2026

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

What is our goal in mirroring all of these across both places? Do we intend to keep them totally in sync?

These examples are pending usage examples that I wanted to wrap up soon. Will remove the link to not have coupled so tightly explicitly. Also, its not meant to duplicate all. Just the most popular ones.

It looks like we've been doing this for a while, but this seems like an antipattern to me

That link was incorrectly linked and will remove.

Or are we just saying that this has the same functionality as the Java version of this template?

Yes, but will remove the link to avoid the confusion.

Btw, is there a reason we're back to blueprint terminology? That may be what is confusing me (#39549 (comment))

Sorry, I started treating anything that is yaml pipeline (with .yaml) as a blueprint regardless of the user. Let me update that to avoid more confusion.

@derrickaw derrickaw Oct 13, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Let me update blueprint to pipelines in the next PR please. Thanks

#40545

@derrickaw
remove template link
1c75f23
@derrickaw
Merge branch 'master' of https://github.com/apache/beam into gcs_avro…
4afa366 
…_to_cloud_bigtable

# Conflicts:
#	sdks/python/apache_beam/yaml/examples/testing/examples_test.py
@derrickaw
derrickaw requested a review from damccorm October 13, 2026 18:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@chamikaramj chamikaramj Awaiting requested review from chamikaramj
@damccorm damccorm Awaiting requested review from damccorm

Assignees

No one assigned

Labels

Next Action: Reviewers python yaml

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@derrickaw @damccorm