-
Notifications
You must be signed in to change notification settings - Fork 15.9k
AIP-84: alias dag_display_name for Job and Xcom
#50065
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
Conversation
Job and Xcomdag_display_name for Job and Xcom
8cdc135 to
606434e
Compare
439f905 to
c205fc4
Compare
jason810496
left a comment
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.
Nice! Thanks for the PR, only one small question about the Job data model.
|
Thanks for the review! |
pierrejeambrun
left a comment
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.
One suggestion, looking good.
Co-authored-by: pierrejeambrun <[email protected]>
90e17f4 to
2d39620
Compare
|
Updated, please take another look. Thanks for all the suggestions and reviews. |
* AIP-84: alias dag_display_name for `Job` and `Xcom` * Remove unused relationship Co-authored-by: pierrejeambrun <[email protected]> * Fix `AliasPath` --------- Co-authored-by: pierrejeambrun <[email protected]>
Related Issue
#46730
cc: @pierrejeambrun @bbovenzi
How
dag_display_nameforJobandXcom^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named
{pr_number}.significant.rstor{issue_number}.significant.rst, in airflow-core/newsfragments.