Skip to content

fix: move template-field validation/transformation out of operato - #70814

Closed
Mr-Neutr0n wants to merge 1 commit into
apache:mainfrom
Mr-Neutr0n:agent/issue-70296-move-template-field-vali
Closed

fix: move template-field validation/transformation out of operato#70814
Mr-Neutr0n wants to merge 1 commit into
apache:mainfrom
Mr-Neutr0n:agent/issue-70296-move-template-field-vali

Conversation

@Mr-Neutr0n

Copy link
Copy Markdown
Contributor

Fixes #70296

Moved GCSFileTransformOperator destination bucket/object defaulting (destination_* or source_*) out of __init__ into execute() and get_openlineage_facets_on_start() so it runs on rendered template values, removed its exemption from validate_operators_init_exemptions.txt, and added test_execute_defaults_destination_to_source (all 3 operator tests pass).

Could not run the suite locally: tlib._bootstrap>:935: in _load_unlocked. Fork CI needs a maintainer approval to run, so this branch has no test signal yet.


This change was prepared with AI assistance under human direction and review.

…f operato

Signed-off-by: Mr-Neutr0n <64578610+Mr-Neutr0n@users.noreply.github.com>
@Mr-Neutr0n

Copy link
Copy Markdown
Contributor Author

Closing this because #70488 already covers GCSFileTransformOperator and is explicitly tracked from #70296. I missed that existing PR before opening this one. This version adds execution and OpenLineage coverage, but competing on the same exemption is not useful, so I will leave the active review on #70488. Sorry for the duplicate.

@Mr-Neutr0n Mr-Neutr0n closed this Jul 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:dev-tools area:providers backport-to-v3-3-test Backport to v3-3-test provider:google Google (including GCP) related issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Move template-field validation/transformation out of operator __init__ (exemption-list burn-down)

1 participant