Skip to content

Validate EmrAddStepsOperator job flow arguments after rendering#70448

Open
haseebmalik18 wants to merge 1 commit into
apache:mainfrom
haseebmalik18:fix-emr-add-steps-template-validation
Open

Validate EmrAddStepsOperator job flow arguments after rendering#70448
haseebmalik18 wants to merge 1 commit into
apache:mainfrom
haseebmalik18:fix-emr-add-steps-template-validation

Conversation

@haseebmalik18

Copy link
Copy Markdown
Contributor

job_flow_id and job_flow_name on EmrAddStepsOperator are template fields, so Jinja renders them after __init__ runs. The mutual-exclusion check lived in __init__, which meant it validated the raw {{ ... }} expressions instead of the actual values. This moves the check into execute() so it runs after rendering.

Part of #70296.


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