Add automated PR watchdog for codelab parameters - #4206
Merged
Conversation
Contributor
|
Note Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported. |
aasthabharill
marked this pull request as draft
September 1, 2026 16:32
aasthabharill
marked this pull request as ready for review
September 1, 2026 18:08
Contributor
|
Note Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported. |
aasthabharill
marked this pull request as draft
September 1, 2026 18:40
aasthabharill
force-pushed
the
codelab-updates2
branch
from
September 1, 2026 18:44
21ebce4 to
e87f833
Compare
aasthabharill
marked this pull request as ready for review
September 2, 2026 08:39
aasthabharill
marked this pull request as draft
September 2, 2026 08:39
Contributor
|
Note Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported. |
Co-authored-by: liferoad <7833268+liferoad@users.noreply.github.com>
…ed connection (#4192) * [SpannerToSourceDb] Reverse replication Integration test with SSL based connection * Moving process creation code into a method * Addressing comments
* add auto-close dependabot step * add ignore comment
* add delta lake to iceberg IT * fix spotless and rerun auto-generation * switch to interrupt call * change to a dynamic import to fix error * add forgotten yaml definition file * add generated README for DeltaLakeToIceberg template * switch back to stop method for direct runner * address gemini comment * spotless * initial comments fixes * change iceberg to lakehouse * change to a wrapped writetoiceberg via writetolakehouse * update yaml file with wrapper transform * remove old writetolakehouse transform idea * spotless * change to renamed transform * address Delta client comment * spotless * fix generated file * create write to lakehouse * change package * oops - used wrong assumed date * fix uri catalog version * update jar to to be the same * change to managed and mimic deltalake * update one more version jar * add manifest resource transformer * add managed and iceberg * change pattern * force to schemaAwareExternalTransform * another try * add transformer * update per Cham's comments * fix formatting
* fix naming * fix default imports issue * rerun some checks * clean up old named files * change pubsubtoiceberg yaml name * update readme with blueprint naming conventions
) * yml file * yml file * changes * Refactor DataStreamToSpannerOptions to separate file * Delete .github/workflows/codelab-watchdog.yml * Refactor SpannerToSourceDbOptions to separate file * mistakes * Refactor GCSSpannerDVOptions to a separate file * spotless * Fix checkstyle formatting and missing package-info * year * year * fix tests * fix tests * testing * spotless * Fix test compilation failures in SpannerToSourceDbTest * spotless
This comment was marked as outdated.
This comment was marked as outdated.
aasthabharill
marked this pull request as ready for review
September 7, 2026 03:34
Contributor
|
Note Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported. |
manitgupta
reviewed
Sep 7, 2026
Member
|
CLA seems to be failing? |
manitgupta
approved these changes
Sep 7, 2026
pratickchokhani
pushed a commit
to pratickchokhani/DataflowTemplates
that referenced
this pull request
Sep 7, 2026
…#4206) * yml file * yml file * changes * deps: update Python dependencies (GoogleCloudPlatform#4194) Co-authored-by: liferoad <7833268+liferoad@users.noreply.github.com> * [SpannerToSourceDb] Reverse replication Integration test with SSL based connection (GoogleCloudPlatform#4192) * [SpannerToSourceDb] Reverse replication Integration test with SSL based connection * Moving process creation code into a method * Addressing comments * add auto-close dependabot step for requirements*.txt type files (GoogleCloudPlatform#4220) * add auto-close dependabot step * add ignore comment * add delta lake to iceberg blueprint and IT (GoogleCloudPlatform#4103) * add delta lake to iceberg IT * fix spotless and rerun auto-generation * switch to interrupt call * change to a dynamic import to fix error * add forgotten yaml definition file * add generated README for DeltaLakeToIceberg template * switch back to stop method for direct runner * address gemini comment * spotless * initial comments fixes * change iceberg to lakehouse * change to a wrapped writetoiceberg via writetolakehouse * update yaml file with wrapper transform * remove old writetolakehouse transform idea * spotless * change to renamed transform * address Delta client comment * spotless * fix generated file * create write to lakehouse * change package * oops - used wrong assumed date * fix uri catalog version * update jar to to be the same * change to managed and mimic deltalake * update one more version jar * add manifest resource transformer * add managed and iceberg * change pattern * force to schemaAwareExternalTransform * another try * add transformer * update per Cham's comments * fix formatting * fix naming determination (GoogleCloudPlatform#4210) * fix naming * fix default imports issue * rerun some checks * clean up old named files * change pubsubtoiceberg yaml name * update readme with blueprint naming conventions * Refactor pipeline parameter options to seperate Options.java file (GoogleCloudPlatform#4209) * yml file * yml file * changes * Refactor DataStreamToSpannerOptions to separate file * Delete .github/workflows/codelab-watchdog.yml * Refactor SpannerToSourceDbOptions to separate file * mistakes * Refactor GCSSpannerDVOptions to a separate file * spotless * Fix checkstyle formatting and missing package-info * year * year * fix tests * fix tests * testing * spotless * Fix test compilation failures in SpannerToSourceDbTest * spotless * change+test * revert test changes --------- Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: liferoad <7833268+liferoad@users.noreply.github.com> Co-authored-by: darshan-sj <darshan-sj@users.noreply.github.com> Co-authored-by: Derrick Williams <derrickaw@google.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Test how this works by playing around with these PRs: