codecov-commenter
commented
2 years ago Codecov Report
Merging #260 (3f65f05) into dev (5caeb6c) will increase coverage by
0.64%. The diff coverage is35.71%.:exclamation: Current head 3f65f05 differs from pull request most recent head c632ca0. Consider uploading reports for the commit c632ca0 to get more accurate results
@@ Coverage Diff @@
## dev #260 +/- ##
==========================================
+ Coverage 43.12% 43.77% +0.64%
==========================================
Files 82 82
Lines 9845 9885 +40
==========================================
+ Hits 4246 4327 +81
+ Misses 5164 5106 -58
- Partials 435 452 +17 | Impacted Files | Coverage Δ | |
|---|---|---|
| pkg/driver/client.go | 0.00% <0.00%> (ø) |
|
| pkg/optimize/optimize_select.go | 69.01% <0.00%> (-6.38%) |
:arrow_down: |
| pkg/optimize/optimizer.go | 40.00% <0.00%> (-4.45%) |
:arrow_down: |
| pkg/plan/query.go | 36.46% <10.00%> (-4.03%) |
:arrow_down: |
| pkg/executor/sharding.go | 46.38% <88.23%> (+46.38%) |
:arrow_up: |
| pkg/plan/delete.go | 7.27% <100.00%> (ø) |
|
| pkg/plan/insert.go | 50.00% <100.00%> (ø) |
|
| pkg/plan/update.go | 10.52% <100.00%> (ø) |
|
| pkg/executor/misc.go | 90.00% <0.00%> (+20.00%) |
:arrow_up: |
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more
ref: https://github.com/cectc/dbpack/issues/
Ⅰ. Describe what this PR did
Ⅱ. Does this pull request fix one issue?
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews