-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added job operator jinja templating #27
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Main points:
- Missing the ability to specify custom template arguments
- Too many fields seem to be template-ized, for all other fields beside the body it's possible to specify their values using Python strings/variables directly. Jinja is overkill for these use cases.
Definitely commented a bit early, I just read a bit more about Airflow's jinja support. Disregard my comment about having templates all over the place. |
@odesenfans Can you please resolve your comments? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Understood the templating mechanism better and all my other comments are fixed.
Note that this PR also improves the support for XCom. Technically you can now pull XCom results and pass them as image, arguments, etc. Good thing because that was my next feature request! |
Fixes