Uh oh!
There was an error while loading. Please reload this page.
Revert adding PhysicalOptimizerRule::optimize_plan - #19186
Merged
Conversation
gabotechs
approved these changes
Dec 7, 2025
adriangb
commented
Dec 7, 2025
ContributorAuthor
@milenkovicm I'll wait for your approval or feedback before merging this |
milenkovicm
approved these changes
Dec 8, 2025
milenkovicm
left a comment
Contributor
There was a problem hiding this comment.
Thanks for understanding @adriangb & @gabotechs
adriangbforce-pushed
the
revert-optimizer-api-change
branch
from
December 8, 2025 15:07
4b62409 to
c986685Compare…e context into optimizer rules (apache#18739)" This reverts commit b990987.
…pache#19030)" This reverts commit 7b4593f.
adriangbforce-pushed
the
revert-optimizer-api-change
branch
from
December 8, 2025 15:49
c986685 to
9b8ff0cCompareadriangb
commented
Dec 8, 2025
ContributorAuthor
(waiting for CI to finish after rebase to merge) |
Uh oh!
There was an error while loading. Please reload this page.
rkrishn7 pushed a commit
to massive-com/arrow-datafusion
that referenced
this pull request
Jan 22, 2026
This reverts apache#18739 by reverting b990987 and 7b4593f. Due to feedback in apache#18739 we've decided to revert it at least for the time being. Since this change has not been released this is not be a breaking API change.
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 freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
3 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.
This reverts #18739 by reverting b990987 and 7b4593f.
Due to feedback in #18739 we've decided to revert it at least for the time being. Since this change has not been released this is not be a breaking API change.