Skip to content
This repository has been archived by the owner on Feb 6, 2024. It is now read-only.

chore: update ceresdbproto & adapter to partition info #124

Merged
merged 2 commits into from
Dec 28, 2022

Conversation

ZuLiangWang
Copy link
Contributor

Which issue does this PR close?

Closes #

Rationale for this change

To support creation of partition tables, we have updated the data structure in ceresdbproto and added encodedPartitionInfo. CeresMeta also needs to adapt to this adjustment.

What changes are included in this PR?

  • Update ceresdbproto
  • Adapt to new data structures.

Are there any user-facing changes?

None.

How does this change test

Pass the existing unit test.

@codecov-commenter
Copy link

codecov-commenter commented Dec 27, 2022

Codecov Report

Merging #124 (8f842ee) into main (423ca15) will decrease coverage by 0.17%.
The diff coverage is 41.68%.

@@            Coverage Diff             @@
##             main     #124      +/-   ##
==========================================
- Coverage   39.80%   39.63%   -0.18%     
==========================================
  Files          29       33       +4     
  Lines        3432     3790     +358     
==========================================
+ Hits         1366     1502     +136     
- Misses       1882     2074     +192     
- Partials      184      214      +30     
Impacted Files Coverage Δ
server/cluster/cluster.go 0.00% <0.00%> (ø)
server/cluster/manager.go 0.00% <0.00%> (ø)
server/cluster/table_manager.go 0.00% <0.00%> (ø)
server/cluster/types.go 0.00% <ø> (ø)
server/coordinator/procedure/create_table.go 0.00% <0.00%> (-53.39%) ⬇️
server/coordinator/procedure/factory.go 0.00% <0.00%> (ø)
server/coordinator/procedure/transfer_leader.go 49.52% <ø> (ø)
server/coordinator/procedure/util.go 38.88% <40.00%> (+0.17%) ⬆️
...er/coordinator/procedure/create_partition_table.go 43.26% <43.26%> (ø)
...erver/coordinator/procedure/create_normal_table.go 46.57% <46.57%> (ø)
... and 4 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@chunshao90
Copy link
Contributor

run go mod tidy.

Copy link
Contributor

@chunshao90 chunshao90 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@chunshao90 chunshao90 merged commit d66bc42 into apache:main Dec 28, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants