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

chore: Fix link to issue and expand comment #14473

Merged
merged 2 commits into from
Feb 4, 2025
Merged

Conversation

findepi
Copy link
Member

@findepi findepi commented Feb 4, 2025

Fix link to issue. Instead of linking to #10500, link to #10553

The link to issue was incorrect.

Also, expand the comment to indicate this is not the final desired state
yet.
@findepi findepi requested a review from alamb February 4, 2025 07:55
@github-actions github-actions bot added logical-expr Logical plan and expressions optimizer Optimizer rules labels Feb 4, 2025
Copy link
Contributor

@alamb alamb left a comment

Choose a reason for hiding this comment

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

Thank you @findepi

@alamb alamb merged commit 0a0ef5c into apache:main Feb 4, 2025
24 of 25 checks passed
@alamb
Copy link
Contributor

alamb commented Feb 4, 2025

Ooops -- I merged this with a CI failure. Fix PR:

@findepi findepi deleted the findepi/cleanup branch February 4, 2025 18:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
logical-expr Logical plan and expressions optimizer Optimizer rules
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants