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

validate behaviour of shadow variables #1267

Open
Licenser opened this issue Oct 18, 2021 · 0 comments
Open

validate behaviour of shadow variables #1267

Licenser opened this issue Oct 18, 2021 · 0 comments
Labels
_complexity:high A task with a high complexity that should be hard _size:medium A medium sized task that will take some time to complete

Comments

@Licenser
Copy link
Member

In many places we just reserve but do not register variables, meaning they're going to be set in the same allocation spot in our local array. This, in theory, could cause problems. Sadly in practice, we've not yet been able to find a script that does turn this theoretical problem into a real problem. We should investigate this. This also touches on modular subqueries as shadow variables are used in the arg replacement (it uses an expression path for this).

@Licenser Licenser added _complexity:high A task with a high complexity that should be hard _size:medium A medium sized task that will take some time to complete labels Nov 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
_complexity:high A task with a high complexity that should be hard _size:medium A medium sized task that will take some time to complete
Projects
None yet
Development

No branches or pull requests

1 participant