hustjieke commented 3 years ago

[summary] support do statement.

[test case] src/proxy/do_test.go sqlparser/parse_test.go

[patch codecov] src/proxy/do.go 100% sqlparser/ast.go 96.7%

codecov[bot] commented 3 years ago

Codecov Report

Merging #721 (81a421a) into master (07f42c3) will increase coverage by 0%. The diff coverage is 100%.

@@          Coverage Diff          @@
##           master   #721   +/-   ##
=====================================
  Coverage      89%    89%           
=====================================
  Files         131    132    +1     
  Lines        9322   9330    +8     
=====================================
+ Hits         8298   8328   +30     
+ Misses        679    663   -16     
+ Partials      345    339    -6

Impacted Files	Coverage Δ
src/proxy/help.go	`100% <ø> (ø)`
src/proxy/do.go	`100% <100%> (ø)`
src/proxy/query.go	`93% <100%> (+6%)`	:arrow_up:
src/backend/queryz.go	`87% <0%> (-4%)`	:arrow_down:
src/proxy/show.go	`82% <0%> (+<1%)`	:arrow_up:

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 07f42c3...81a421a. Read the comment docs.

hustjieke commented 3 years ago

The codecov 62% is not related to this patch.

hustjieke commented 3 years ago

@andyli029 Please make a review, thanks!

BohuTANG commented 3 years ago

codecov CI failed

hustjieke commented 3 years ago

codecov CI failed

I complete the code coverage of query.go and now jthe codecov is passed.

andyli029 commented 3 years ago

LGTM

radondb / radon

*.: support do stmt #720 #721

Codecov Report