Skip to content

UI: Expose list filters the API already supported - #74764

Open
bbovenzi wants to merge 2 commits into
apache:mainfrom
astronomer:feat-missing-table-filters
Open

bbovenzi wants to merge 2 commits into
apache:mainfrom
astronomer:feat-missing-table-filters

Conversation

@bbovenzi

@bbovenzi bbovenzi commented Oct 14, 2026 •

Copy link
Copy Markdown
Contributor

Add filters for table elements that already had backend support:

Task Instance:

  • Run After instead of Logical Date

Asset Events:

  • Source Run Id
  • Source Map Index
  • Partition Key

Backfills:

  • Running / Completed

Dags:

  • Has import errors

Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)
    Claude Opus 5

  • Read the Pull Request Guidelines for more information. Note: commit author/co-author name and email in commits become permanently public when merged.
  • For fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
  • When adding dependency, check compliance with the ASF 3rd Party License Policy.
  • For significant user-facing changes create newsfragment: {pr_number}.significant.rst, in airflow-core/newsfragments. You can add this file in a follow-up commit after the PR is created so you know the PR number.

@yuseok89 yuseok89 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks! I left one question about existing logical-date links inline.

@pierrejeambrun pierrejeambrun left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks, nice to see these wired up since the API already supported them. I think this needs a rebase on main first, it conflicts with the task loop changes. While testing locally I found that the new Executor and Run After pills don't render their values on the Task Instances page, details inline, along with a couple of other things.

Comment thread airflow-core/src/airflow/ui/src/pages/TaskInstances/TaskInstancesFilter.tsx Outdated
Comment thread airflow-core/src/airflow/ui/src/constants/filterConfigs.tsx Outdated
@bbovenzi
bbovenzi force-pushed the feat-missing-table-filters branch from 21b890f to e31593f Compare October 20, 2026 20:28
Several columns on the Dags, Task Instances, Asset Events and Backfills
tables could be read but not filtered on, even though the endpoints had
accepted the parameters all along. Each pill added here maps to one that
already existed, so none of this needed an API change.

Task Instances gains Run After alongside the Logical Date range it already
offered. The two are not interchangeable: the table shows run_after, and
logical_date is null for manual and asset-triggered runs, so filtering by
it silently drops those. Both are offered so either question can be asked,
and existing logical_date links keep working unchanged.

Co-Authored-By: Claude <noreply@anthropic.com>
@bbovenzi
bbovenzi force-pushed the feat-missing-table-filters branch from e31593f to 3f90a2d Compare October 21, 2026 13:32

@pierrejeambrun pierrejeambrun left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, description can be updated (executor shouldn't be mentioned anymore)

Comment thread airflow-core/src/airflow/ui/src/pages/DagsList/DagsList.tsx
Co-authored-by: Sean Ghaeli <58916776+seanghaeli@users.noreply.github.com>
@bbovenzi
bbovenzi enabled auto-merge (squash) October 29, 2026 21:18

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:translations area:UI Related to UI/UX. For Frontend Developers. translation:default

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants