Skip to content

Support the new max_attempts and max_splits in the sequence file export.#3569

Open
justinuang wants to merge 1 commit intoGoogleCloudPlatform:mainfrom
justinuang:Dataflow_Improvements
Open

Support the new max_attempts and max_splits in the sequence file export.#3569
justinuang wants to merge 1 commit intoGoogleCloudPlatform:mainfrom
justinuang:Dataflow_Improvements

Conversation

@justinuang
Copy link
Copy Markdown
Contributor

@justinuang justinuang commented Mar 26, 2026

This PR adds support for the bigtableMaxAttempts parameter to the Cloud_Bigtable_to_GCS_SequenceFile Dataflow template. This is the Dataflow side of the implementation, exposing the parameter in the template metadata so it can be passed to the underlying ExportJob (which already supports it via the java-bigtable-hbase library).

@justinuang justinuang changed the title Support the new settings in the sequence file export. Support the new max_attempts and max_splits in the sequence file export. Mar 26, 2026
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 26, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 52.17%. Comparing base (2c14d1f) to head (747d460).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@              Coverage Diff              @@
##               main    #3569       +/-   ##
=============================================
+ Coverage     37.00%   52.17%   +15.17%     
- Complexity      470     5647     +5177     
=============================================
  Files           194     1040      +846     
  Lines         11376    63074    +51698     
  Branches       1127     6916     +5789     
=============================================
+ Hits           4210    32912    +28702     
- Misses         6831    27930    +21099     
- Partials        335     2232     +1897     
Components Coverage Δ
spanner-templates 72.24% <ø> (∅)
spanner-import-export 69.07% <ø> (∅)
spanner-live-forward-migration 80.47% <ø> (∅)
spanner-live-reverse-replication 77.87% <ø> (∅)
spanner-bulk-migration 89.18% <ø> (∅)
gcs-spanner-dv 85.32% <ø> (∅)
Files with missing lines Coverage Δ
...cloud/teleport/templates/ExportJobPlaceholder.java 0.00% <ø> (ø)

... and 868 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.

@justinuang justinuang force-pushed the Dataflow_Improvements branch from e3c7b75 to 747d460 Compare March 26, 2026 19:43
@gemini-code-assist
Copy link
Copy Markdown

Warning

Gemini encountered an error creating the summary. You can try again by commenting /gemini summary.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant