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

Jump to bottom

planner: use SEMI_JOIN_REWRITE hint to rewrite the semi join #35325

Merged
merged 20 commits into from
Jul 14, 2022
Merged

planner: use SEMI_JOIN_REWRITE hint to rewrite the semi join #35325

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
5b190ef
planner: use SEMI_JOIN_REWRITE hint to rewrite the semi join
winoros Jun 13, 2022
5f004c3
add tests
winoros Jun 13, 2022
a8467b0
add the wrongly not added file
winoros Jun 13, 2022
088360c
Merge branch 'master' into semi-join-rewrite-with-hint
hawkingrei Jun 14, 2022
6e73b42
Merge branch 'master' into semi-join-rewrite-with-hint
winoros Jun 28, 2022
ba43f75
fix the bug found by AilinKid
winoros Jun 29, 2022
4b82ca3
Merge remote-tracking branch 'yiding/semi-join-rewrite-with-hint' int…
winoros Jun 29, 2022
71ff868
Merge branch 'master' into semi-join-rewrite-with-hint
AilinKid Jun 29, 2022
c563356
add tests
winoros Jun 29, 2022
78de4de
Merge remote-tracking branch 'yiding/semi-join-rewrite-with-hint' int…
winoros Jun 29, 2022
9670a0f
fix license header
winoros Jun 29, 2022
332adef
fix typo
winoros Jun 29, 2022
e5ee1ae
Merge branch 'master' into semi-join-rewrite-with-hint
AilinKid Jun 30, 2022
ded3884
Merge branch 'master' into semi-join-rewrite-with-hint
AilinKid Jul 7, 2022
796ecbb
Merge branch 'master' into semi-join-rewrite-with-hint
winoros Jul 12, 2022
dfabf99
fix test
winoros Jul 12, 2022
3bc72a8
Merge remote-tracking branch 'yiding/semi-join-rewrite-with-hint' int…
winoros Jul 12, 2022
0397da9
fix comment
winoros Jul 12, 2022
385184e
Merge branch 'master' into semi-join-rewrite-with-hint
ti-chi-bot Jul 14, 2022
ab37783
Merge branch 'master' into semi-join-rewrite-with-hint
ti-chi-bot Jul 14, 2022
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
23 changes: 23 additions & 0 deletions bindinfo/bind_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -331,6 +331,29 @@ func TestExplain(t *testing.T) {
tk.MustExec("drop global binding for SELECT * from t1 union SELECT * from t1")
}

func TestBindSemiJoinRewrite(t *testing.T) {
store, clean := testkit.CreateMockStore(t)
defer clean()

tk := testkit.NewTestKit(t, store)
tk.MustExec("use test")
tk.MustExec("drop table if exists t1")
tk.MustExec("drop table if exists t2")
tk.MustExec("create table t1(id int)")
tk.MustExec("create table t2(id int)")
require.True(t, tk.HasKeywordInOperatorInfo("select * from t1 where exists(select 1 from t2 where t1.id=t2.id)", "semi join"))
require.True(t, tk.NotHasKeywordInOperatorInfo("select * from t1 where exists(select /*+ SEMI_JOIN_REWRITE() */ 1 from t2 where t1.id=t2.id)", "semi join"))

tk.MustExec(`
create global binding for
select * from t1 where exists(select 1 from t2 where t1.id=t2.id)
using
select * from t1 where exists(select /*+ SEMI_JOIN_REWRITE() */ 1 from t2 where t1.id=t2.id)
`)

require.True(t, tk.NotHasKeywordInOperatorInfo("select * from t1 where exists(select 1 from t2 where t1.id=t2.id)", "semi join"))
}

// TestBindingSymbolList tests sql with "?, ?, ?, ?", fixes #13871
func TestBindingSymbolList(t *testing.T) {
store, dom, clean := testkit.CreateMockStoreAndDomain(t)
Expand Down
Loading