plan: new plan support union. #3085
Conversation
plan/dag_plan_test.go
Outdated
| // Test complex update. | ||
| { | ||
| sql: "update t set a = 5 where b < 1 order by d limit 1", | ||
| best: "TableReader(Table(t)->Sel([lt(test.t.b, 1)])->Limit)->Limit->*plan.Update", |
There was a problem hiding this comment.
There is a bug in old planner ...
mysql> create table t1(a int primary key,b int);
Query OK, 0 rows affected (0.01 sec)
mysql> explain update t1 set a = 1 where a < 1 order by b limit 1;
+-------------+-------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+----------+
| ID | Json | ParentID |
+-------------+-------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+----------+
| TableScan_6 | {
"db": "test",
"table": "t1",
"desc": false,
"keep order": true,
"push down info": {
"limit": 1,
"access conditions": [
"lt(test.t1.a, 1)"
],
"index filter conditions": null,
"table filter conditions": null
}
} | Update_5 |
| Update_5 | {
"children": [
"TableScan_6"
]
} | |
+-------------+-------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+----------+
2 rows in set (0.00 sec)
plan/dag_plan_test.go
Outdated
| // Test complex delete. | ||
| { | ||
| sql: "delete from t where b < 1 order by d limit 1", | ||
| best: "TableReader(Table(t)->Sel([lt(test.t.b, 1)])->Limit)->Limit->*plan.Delete", |
|
LGTM |
winoros
added
status/LGT2
new plan support union all.
@shenli @coocood @zimulala @winoros PTAL