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

Fix inline functions substituting parameters from hidden nodes. #8288

Merged
merged 2 commits into from
Feb 4, 2025

Conversation

dnwpark
Copy link
Contributor

@dnwpark dnwpark commented Feb 3, 2025

close #8285

A function body may be a statement which contains references to a parent statement. Ensure that this parent's parameters are not substituted while inlining the function parameters.

@dnwpark dnwpark requested review from mmastrac, aljazerzen and msullivan and removed request for mmastrac February 3, 2025 20:11
@dnwpark dnwpark marked this pull request as ready for review February 3, 2025 20:11
@dnwpark dnwpark force-pushed the dont-inline-in-hidden branch from 64b1128 to 9b5ae32 Compare February 3, 2025 21:37
@msullivan
Copy link
Member

Mark to-backport-6.x?

@dnwpark dnwpark added the to-backport-6.x PRs that *should* be backported to 6.x label Feb 4, 2025
@dnwpark dnwpark merged commit e4a6db4 into master Feb 4, 2025
23 checks passed
@dnwpark dnwpark deleted the dont-inline-in-hidden branch February 4, 2025 15:04
dnwpark added a commit that referenced this pull request Feb 4, 2025
A function body may be a statement which contains references to a parent statement. Ensure that this parent's parameters are not substituted while inlining the function parameters.
@msullivan msullivan added backported-6.x PRs that *have* been backported to 6.x and removed to-backport-6.x PRs that *should* be backported to 6.x labels Feb 4, 2025
msullivan pushed a commit that referenced this pull request Feb 4, 2025
A function body may be a statement which contains references to a parent statement. Ensure that this parent's parameters are not substituted while inlining the function parameters.
msullivan pushed a commit that referenced this pull request Feb 4, 2025
A function body may be a statement which contains references to a parent statement. Ensure that this parent's parameters are not substituted while inlining the function parameters.
deepbuzin pushed a commit that referenced this pull request Feb 18, 2025
A function body may be a statement which contains references to a parent statement. Ensure that this parent's parameters are not substituted while inlining the function parameters.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backported-6.x PRs that *have* been backported to 6.x
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cryptic error when running insert queries within functions
3 participants