radondb / radon

RadonDB is an open source, cloud-native MySQL database for building global, scalable cloud services
https://radondb.io/
GNU General Public License v3.0
1.8k stars 217 forks source link

*.: complete delete syntax #713 #714

Closed hustjieke closed 3 years ago

hustjieke commented 3 years ago

[summary]

  1. complete single-table syntax, add delete options and parition.
  2. add support to parse multi-table syntax.
  3. add support to delete without where clause, like "delete from t1 limit 2"

[test case] sqlparser/ast_test.go sqlparser/parse_test.go sqlparser/ast_test.go src/planner/builder/expr_test.go src/planner/delete_plan_test.go

[patch codecov] sqlparser/ast.go 96.1% sqlparser/ast_funcs.go 94.4% src/planner/delete_plan.go 93.9% src/planner/update_plan.go 95.1%

codecov[bot] commented 3 years ago

Codecov Report

Merging #714 (7bf6cd6) into master (9c0186b) will decrease coverage by 0%. The diff coverage is 92%.

Impacted file tree graph

@@          Coverage Diff          @@
##           master   #714   +/-   ##
=====================================
- Coverage      89%    89%   -1%     
=====================================
  Files         133    133           
  Lines        9406   9464   +58     
=====================================
+ Hits         8402   8453   +51     
- Misses        664    669    +5     
- Partials      340    342    +2     
Impacted Files Coverage Δ
src/backend/queryz.go 90% <ø> (+3%) :arrow_up:
src/planner/builder/expr.go 98% <ø> (ø)
src/planner/common.go 92% <90%> (-8%) :arrow_down:
src/planner/delete_plan.go 93% <93%> (+1%) :arrow_up:
src/planner/update_plan.go 92% <100%> (ø)
src/backend/xacheck.go 80% <0%> (-2%) :arrow_down:
src/backend/txn.go 86% <0%> (-1%) :arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 9c0186b...7bf6cd6. Read the comment docs.

hustjieke commented 3 years ago

@andyli029 Please make a review, thanks!

andyli029 commented 3 years ago

LGTM

BohuTANG commented 3 years ago

LGTM