Skip to content

Commit

Permalink
modify unit test expected best plan, because empty range would
Browse files Browse the repository at this point in the history
be converted to TableDual now
  • Loading branch information
eurekaka committed Sep 3, 2018
1 parent 0a603d1 commit b1c1b24
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions plan/physical_plan_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,7 @@ func (s *testPlanSuite) TestDAGPlanBuilderSimpleCase(c *C) {
},
{
sql: "select * from t where (t.c > 0 and t.c < 1) or (t.c > 2 and t.c < 3) or (t.c > 4 and t.c < 5) or (t.c > 6 and t.c < 7) or (t.c > 9 and t.c < 10)",
best: "IndexLookUp(Index(t.c_d_e)[], Table(t))",
best: "Dual",
},
// Test TopN to table branch in double read.
{
Expand All @@ -87,7 +87,7 @@ func (s *testPlanSuite) TestDAGPlanBuilderSimpleCase(c *C) {
// Test Null Range but the column has not null flag.
{
sql: "select * from t where t.c is null",
best: "IndexLookUp(Index(t.c_d_e)[], Table(t))",
best: "Dual",
},
// Test TopN to index branch in double read.
{
Expand Down Expand Up @@ -1025,7 +1025,7 @@ func (s *testPlanSuite) TestRefine(c *C) {
},
{
sql: "select a from t where c in (1, 2, 3) and (d > 3 and d < 4 or d > 5 and d < 6)",
best: "IndexReader(Index(t.c_d_e)[])->Projection",
best: "Dual->Projection",
},
{
sql: "select a from t where c in (1, 2, 3) and (d > 2 and d < 4 or d > 5 and d < 7)",
Expand Down

0 comments on commit b1c1b24

Please sign in to comment.