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.79k stars 218 forks source link

*: fix the bug that the condition after show variables does not take … #731

Closed zhyass closed 3 years ago

zhyass commented 3 years ago

…effect #730

[summary]

  1. Remove force_eof in some statement.
  2. Completing show statement syntax.

[test case] src/vendor/github.com/xelabs/go-mysqlstack/sqlparser/show_test.go

[patch codecov] src/vendor/github.com/xelabs/go-mysqlstack/sqlparser/ast.go 96.5% src/vendor/github.com/xelabs/go-mysqlstack/sqlparser/sql.go 92.0%

codecov[bot] commented 3 years ago

Codecov Report

Merging #731 (86c0b24) into master (e71eeff) will decrease coverage by 0%. The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff          @@
##           master   #731   +/-   ##
=====================================
- Coverage      89%    89%   -1%     
=====================================
  Files         133    133           
  Lines        9478   9478           
=====================================
- Hits         8475   8473    -2     
- Misses        665    666    +1     
- Partials      338    339    +1     
Impacted Files Coverage Δ
src/proxy/query.go 93% <ø> (ø)
src/backend/txn.go 87% <0%> (-1%) :arrow_down:
src/executor/engine/merge_join.go 96% <0%> (ø)

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 e71eeff...86c0b24. Read the comment docs.

BohuTANG commented 3 years ago

LGTM