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

Set udv allow more expressions #6964

Merged
merged 7 commits into from
Nov 2, 2020

Conversation

harshit-gangal
Copy link
Member

@harshit-gangal harshit-gangal commented Oct 28, 2020

Fixes: #6915

Set UDV allowed only specific function calls. With this change. UDV can be set using any function all and can also be done through subquery.

Signed-off-by: Harshit Gangal <harshit@planetscale.com>
Signed-off-by: Harshit Gangal <harshit@planetscale.com>
Signed-off-by: Harshit Gangal <harshit@planetscale.com>
Signed-off-by: Harshit Gangal <harshit@planetscale.com>
Signed-off-by: Harshit Gangal <harshit@planetscale.com>
Signed-off-by: Harshit Gangal <harshit@planetscale.com>
Signed-off-by: Harshit Gangal <harshit@planetscale.com>
@harshit-gangal harshit-gangal marked this pull request as ready for review October 29, 2020 18:50
@deepthi
Copy link
Member

deepthi commented Oct 29, 2020

Please write a PR description before merging.

@harshit-gangal harshit-gangal changed the title Set udv allow more Set udv allow more expressions. Nov 2, 2020
@harshit-gangal harshit-gangal changed the title Set udv allow more expressions. Set udv allow more expressions Nov 2, 2020
@harshit-gangal harshit-gangal merged commit b2433cc into vitessio:master Nov 2, 2020
@harshit-gangal harshit-gangal deleted the set-udv-allow-more branch November 2, 2020 06:30
@askdba askdba added this to the v9.0 milestone Nov 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Can't set user defined variable to non-trivial expressions
4 participants