Skip to content

Move SnowparkContainerJobOperator template-field validation to execute - #70332

Merged
shahar1 merged 1 commit into
apache:mainfrom
justinpakzad:fix/init-validation-snowpark-container-op
Jul 24, 2026
Merged

Move SnowparkContainerJobOperator template-field validation to execute#70332
shahar1 merged 1 commit into
apache:mainfrom
justinpakzad:fix/init-validation-snowpark-container-op

Conversation

@justinpakzad

Copy link
Copy Markdown
Contributor

This PR moves the validation of templated fields in the SnowparkContainerJobOperator from __init__ to theexecute() method. The existing tests were updated accordingly and the class was removed from the exemptions txt file.

related: #70296

Was generative AI tooling used to co-author this PR?
  • No (please specify the tool below)

  • Read the Pull Request Guidelines for more information. Note: commit author/co-author name and email in commits become permanently public when merged.
  • For fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
  • When adding dependency, check compliance with the ASF 3rd Party License Policy.
  • For significant user-facing changes create newsfragment: {pr_number}.significant.rst, in airflow-core/newsfragments. You can add this file in a follow-up commit after the PR is created so you know the PR number.

@shahar1 shahar1 left a comment

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.

Great!

@shahar1
shahar1 merged commit 8751c75 into apache:main Jul 24, 2026
83 checks passed
@shahar1 shahar1 removed the backport-to-v3-3-test Backport to v3-3-test label Jul 24, 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 provider:snowflake Issues related to Snowflake provider

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants