Repository navigation
[v3-1-test] remove large in clause in assets with cte and join (#62114) - #63511
Merged
eladkal merged 5 commits intoMar 13, 2026
Merged
Conversation
…e#62114)\n\n* fixed large in clause\n\n* fixed tests\n\n* changed to delete using\n\n* added compatibility for tests\n\n* Change asset selection to use CTE\n\n* Fix asset selection for Airflow versions\n\n* fixed some tests and optimized the query\n\n* fixed all tests\n\n* fixed mypy\n\n* fixup! address cr comments\n\n* address CR comments, added tests and fixed plugin\n\n* adress CR comment\n\n---------\n(cherry picked from commit a89dcaa)\n\nCo-authored-by: Nataneljpwd <104094207+Nataneljpwd@users.noreply.github.com>\nCo-authored-by: Natanel Rudyuklakir <natanelrudyuklakir@gmail.com>\nCo-authored-by: Elad Kalif 45845474+eladkal@users.noreply.github.com>
1 of 2 tasks
eladkal
approved these changes
Mar 13, 2026
vatsrahul1001
pushed a commit
that referenced
this pull request
Mar 25, 2026
… (#63511) * [v3-1-test] remove large in clause in assets with cte and join (#62114)\n\n* fixed large in clause\n\n* fixed tests\n\n* changed to delete using\n\n* added compatibility for tests\n\n* Change asset selection to use CTE\n\n* Fix asset selection for Airflow versions\n\n* fixed some tests and optimized the query\n\n* fixed all tests\n\n* fixed mypy\n\n* fixup! address cr comments\n\n* address CR comments, added tests and fixed plugin\n\n* adress CR comment\n\n---------\n(cherry picked from commit a89dcaa)\n\nCo-authored-by: Nataneljpwd <104094207+Nataneljpwd@users.noreply.github.com>\nCo-authored-by: Natanel Rudyuklakir <natanelrudyuklakir@gmail.com>\nCo-authored-by: Elad Kalif 45845474+eladkal@users.noreply.github.com> * fix CTE import * set execution options * lint * fixed ruff --------- Co-authored-by: Natanel Rudyuklakir <natanelrudyuklakir@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
[v3-1-test] remove large in clause in assets with cte and join (#62114)
fixed large in clause
fixed tests
changed to delete using
added compatibility for tests
Change asset selection to use CTE
Fix asset selection for Airflow versions
fixed some tests and optimized the query
fixed all tests
fixed mypy
fixup! address cr comments
address CR comments, added tests and fixed plugin
adress CR comment
---------(cherry picked from commit a89dcaa)
Co-authored-by: Nataneljpwd 104094207+Nataneljpwd@users.noreply.github.com
Co-authored-by: Natanel Rudyuklakir natanelrudyuklakir@gmail.com
Co-authored-by: Elad Kalif 45845474+eladkal@users.noreply.github.com>
Was generative AI tooling used to co-author this PR?
{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.