Skip to content
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

Add more filters and search to get_dags endpoint #42321

Merged
merged 2 commits into from
Sep 24, 2024

Conversation

pierrejeambrun
Copy link
Member

@pierrejeambrun pierrejeambrun commented Sep 18, 2024

Related to: #42159

PR based on #42320 that adds:

filters:

  • owners

sort:

  • dag_display_name
  • next_dagrun

search:

  • dag_display_name_pattern

Problem encountered:

  • Mysql no support for nullslast
  • Different ordering of True / False based on database implementation

@pierrejeambrun pierrejeambrun added the AIP-84 Modern Rest API label Sep 18, 2024
@pierrejeambrun pierrejeambrun self-assigned this Sep 18, 2024
@pierrejeambrun pierrejeambrun changed the title Add dag display name order by Add more filters and search Sep 18, 2024
@pierrejeambrun pierrejeambrun force-pushed the add-dag-display-name-filter branch 2 times, most recently from 34624ad to d25f3e0 Compare September 18, 2024 16:44
@pierrejeambrun pierrejeambrun force-pushed the add-dag-display-name-filter branch 5 times, most recently from c460f6a to e4ad7f3 Compare September 23, 2024 13:23
@pierrejeambrun pierrejeambrun changed the title Add more filters and search Add more filters and search to get_dags= Sep 23, 2024
@pierrejeambrun pierrejeambrun changed the title Add more filters and search to get_dags= Add more filters and search to get_dags endpoint Sep 23, 2024
@pierrejeambrun pierrejeambrun merged commit e0bddbc into apache:main Sep 24, 2024
51 checks passed
@pierrejeambrun pierrejeambrun deleted the add-dag-display-name-filter branch September 24, 2024 09:24
joaopamaral pushed a commit to joaopamaral/airflow that referenced this pull request Oct 21, 2024
* Add more search and filters

* Fix MySQL syntax error
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AIP-84 Modern Rest API
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants