Skip to content

AIP-84 Add ordering to task instances and mapped task instances list#45802

Merged
pierrejeambrun merged 9 commits intoapache:mainfrom
astronomer:add_task_instance_ordering
Jan 23, 2025
Merged

AIP-84 Add ordering to task instances and mapped task instances list#45802
pierrejeambrun merged 9 commits intoapache:mainfrom
astronomer:add_task_instance_ordering

Conversation

@prabhusneha
Copy link
Contributor

Closes: #43778

Added additional ordering options to list task instance and mapped task instances.

Copy link
Member

@pierrejeambrun pierrejeambrun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice.

I think the original comment was also referring to data_interval_start/end were you able to take a look ?

Also some test would be great. Not all sort params are tested for task instances, but that wouldn't hurt.

Copy link
Member

@pierrejeambrun pierrejeambrun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, just a few minor adjustments and we should be good to merge.

Thanks 🎉

Copy link
Member

@pierrejeambrun pierrejeambrun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good, thanks!

@pierrejeambrun pierrejeambrun added the AIP-84 Modern Rest API label Jan 23, 2025
@pierrejeambrun pierrejeambrun merged commit f01c53a into apache:main Jan 23, 2025
46 checks passed
@pierrejeambrun pierrejeambrun deleted the add_task_instance_ordering branch January 23, 2025 09:08
dauinh pushed a commit to dauinh/airflow that referenced this pull request Jan 24, 2025
…pache#45802)

* add attributes to task instance ordering

* add attributes to task instance ordering

* add additional ordering attributes and add test

* add test for mapped task instances, parametrized the tests

---------

Co-authored-by: Sneha Prabhu <snehaprabhu@Snehas-MacBook-Pro.local>
@utkarsharma2 utkarsharma2 added the type:new-feature Changelog: New Features label Jan 27, 2025
@utkarsharma2 utkarsharma2 added this to the Airflow 3.0.0 milestone Jan 27, 2025
gpathak128 pushed a commit to gpathak128/airflow that referenced this pull request Jan 29, 2025
…pache#45802)

* add attributes to task instance ordering

* add attributes to task instance ordering

* add additional ordering attributes and add test

* add test for mapped task instances, parametrized the tests

---------

Co-authored-by: Sneha Prabhu <snehaprabhu@Snehas-MacBook-Pro.local>
got686-yandex pushed a commit to got686-yandex/airflow that referenced this pull request Jan 30, 2025
…pache#45802)

* add attributes to task instance ordering

* add attributes to task instance ordering

* add additional ordering attributes and add test

* add test for mapped task instances, parametrized the tests

---------

Co-authored-by: Sneha Prabhu <snehaprabhu@Snehas-MacBook-Pro.local>
niklasr22 pushed a commit to niklasr22/airflow that referenced this pull request Feb 8, 2025
…pache#45802)

* add attributes to task instance ordering

* add attributes to task instance ordering

* add additional ordering attributes and add test

* add test for mapped task instances, parametrized the tests

---------

Co-authored-by: Sneha Prabhu <snehaprabhu@Snehas-MacBook-Pro.local>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AIP-84 Modern Rest API type:new-feature Changelog: New Features

Projects

No open projects

Development

Successfully merging this pull request may close these issues.

AIP-84 Add ordering to task instances and mapped task instances list

3 participants