codecov-commenter
commented
1 year ago Codecov Report
Merging #131 (05ee7e0) into main (494e15e) will decrease coverage by
0.08%. The diff coverage is39.08%.
@@ Coverage Diff @@
## main #131 +/- ##
==========================================
- Coverage 39.07% 38.99% -0.09%
==========================================
Files 33 33
Lines 4217 4234 +17
==========================================
+ Hits 1648 1651 +3
- Misses 2317 2331 +14
Partials 252 252 | Impacted Files | Coverage Δ | |
|---|---|---|
| server/cluster/cluster.go | 0.00% <0.00%> (ø) |
|
| server/cluster/table_manager.go | 0.00% <0.00%> (ø) |
|
| server/cluster/types.go | 0.00% <0.00%> (ø) |
|
| ...er/coordinator/procedure/create_partition_table.go | 39.11% <9.67%> (-1.40%) |
:arrow_down: |
| server/coordinator/procedure/create_table.go | 46.23% <50.00%> (ø) |
|
| server/storage/types.go | 70.10% <86.66%> (-0.45%) |
:arrow_down: |
| .../coordinator/procedure/create_table_common_util.go | 70.49% <100.00%> (ø) |
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.
Which issue does this PR close?
Closes #
Rationale for this change
In order to support the case where the partition table is opened by multiple machines in a non-cloud environment, store the partition info of
PartitionTableinceresmeta.What changes are included in this PR?
PartitionInfoin metadata of the table.Are there any user-facing changes?
No.
How does this change test
Manual testing.