Skip to content

AIP-38 Invalidate DryRun query cache on submit - #46238

Merged
bbovenzi merged 1 commit into
apache:mainfrom
astronomer:invalidate-dry-run-query-cache-on-submit
Jan 29, 2025
Merged

AIP-38 Invalidate DryRun query cache on submit#46238
bbovenzi merged 1 commit into
apache:mainfrom
astronomer:invalidate-dry-run-query-cache-on-submit

Conversation

@pierrejeambrun

Copy link
Copy Markdown
Member

When we submit a clear or a marks as action, we should invalidate the associated DryRun query. Otherwise the modal displaying the affected task instance table will keep showing old responses after a successful submit.

To reproduce the issue, you can clear a task instance and then

@pierrejeambrunpierrejeambrun added the AIP-38 Modern Web Application label Jan 29, 2025
@pierrejeambrunpierrejeambrun added this to the Airflow 3.0.0 milestone Jan 29, 2025
@pierrejeambrunpierrejeambrun self-assigned this Jan 29, 2025
@boring-cyborgboring-cyborgBot added the area:UI Related to UI/UX. For Frontend Developers. label Jan 29, 2025
@pierrejeambrun
pierrejeambrunforce-pushed the invalidate-dry-run-query-cache-on-submit branch from c3621cd to 6fc1eb9CompareJanuary 29, 2025 13:54
@bbovenzi

Copy link
Copy Markdown
Contributor

lgtm but looks like some db tests are unhappy

@pierrejeambrun

Copy link
Copy Markdown
MemberAuthor

lgtm but looks like some db tests are unhappy

Yes main is broken for some other reason. I'll wait for #46245 that should fix them.

@shubhamraj-gitshubhamraj-git 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.

+1 LGTM post main gets fixed.

@bbovenzi
bbovenziforce-pushed the invalidate-dry-run-query-cache-on-submit branch from 6fc1eb9 to 0ec70e1CompareJanuary 29, 2025 19:27
@bbovenzi
bbovenzi merged commit 4a9accf into apache:mainJan 29, 2025
@bbovenzi
bbovenzi deleted the invalidate-dry-run-query-cache-on-submit branch January 29, 2025 20:30
got686-yandex pushed a commit to got686-yandex/airflow that referenced this pull request Jan 30, 2025
dabla pushed a commit to dabla/airflow that referenced this pull request Jan 30, 2025
ambika-garg pushed a commit to ambika-garg/airflow that referenced this pull request Jan 30, 2025
niklasr22 pushed a commit to niklasr22/airflow that referenced this pull request Feb 8, 2025
ambika-garg pushed a commit to ambika-garg/airflow that referenced this pull request Feb 17, 2025
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AIP-38Modern Web Applicationarea:UIRelated to UI/UX. For Frontend Developers.

Projects

No open projects

Development

Successfully merging this pull request may close these issues.

3 participants

@pierrejeambrun@bbovenzi@shubhamraj-git