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.

Sign up for GitHub

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

fixes bugs around expression precedence and LIKE #16934

Merged
merged 15 commits into from
Oct 14, 2024
Merged

fixes bugs around expression precedence and LIKE #16934

Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
e5b4135
fixes precedence of NOT LIKE
systay Oct 11, 2024
b7662bd
update test expectations
systay Oct 11, 2024
fd3ecaa
remove JSONExtract as binary op. Just keep it as a function call
systay Oct 11, 2024
8d3825a
parse ->> as json_unquote(json_extract()) to simplify precendence logic
systay Oct 11, 2024
6d5c269
update test expectations
systay Oct 11, 2024
46f7bd5
remove unused constants
systay Oct 11, 2024
b0b93d9
add more tests for LIKE in the evalengine
systay Oct 11, 2024
6ffb2e0
bugfix: two fixes for (NOT) LIKE
systay Oct 11, 2024
ab3d000
bugfix: don't trust SQLTypes
systay Oct 11, 2024
f1b52ea
add end2end test with problematic queries
systay Oct 11, 2024
c38c368
wip - another attempt at the evalengine fix
systay Oct 11, 2024
867bc95
make sure to handle eval and compilation similarly for LIKE
systay Oct 12, 2024
9efcf48
clean up code
systay Oct 14, 2024
bd67c87
feat: fix locate null check escaping
GuptaManan100 Oct 14, 2024
02fe1f2
test: simplify code
systay Oct 14, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
update test expectations 
Signed-off-by: Andres Taylor <andres@planetscale.com>
  • Loading branch information
@systay
systay committed Oct 11, 2024
commit 6d5c269e0e58ed2556ce0e1e187e6da59e3ed71a
4 changes: 2 additions & 2 deletions go/vt/vtgate/planbuilder/testdata/select_cases.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3603,8 +3603,8 @@
"Name": "user",
"Sharded": true
},
"FieldQuery": "select a -> '$[4]', a ->> '$[3]' from `user` where 1 != 1",
"Query": "select a -> '$[4]', a ->> '$[3]' from `user`",
"FieldQuery": "select json_extract(a, '$[4]'), json_unquote(json_extract(a, '$[3]')) from `user` where 1 != 1",
"Query": "select json_extract(a, '$[4]'), json_unquote(json_extract(a, '$[3]')) from `user`",
"Table": "`user`"
},
"TablesUsed": [
Expand Down
Loading