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

feat: add deploy mode #163

Merged
merged 2 commits into from
Apr 27, 2023
Merged

feat: add deploy mode #163

merged 2 commits into from
Apr 27, 2023

Conversation

ZuLiangWang
Copy link
Contributor

Which issue does this PR close?

Closes #

Rationale for this change

We support automatic failover and load balancing in the latest version, but they are not applicable in some scenarios, so add a switch to control their opening.

What changes are included in this PR?

  • Add DeployMode config, When the deployMode is turned on, the failover scheduling will be turned off, which is used for CeresDB cluster publishing and using local storage.

Are there any user-facing changes?

None.

How does this change test

Pass all unit tests and verify that the switch takes effect in the local environment.

server/config/config.go Outdated Show resolved Hide resolved
server/config/config.go Outdated Show resolved Hide resolved
server/storage/types.go Show resolved Hide resolved
server/service/http/api.go Outdated Show resolved Hide resolved
server/coordinator/scheduler/scheduler_manager.go Outdated Show resolved Hide resolved
server/coordinator/scheduler/scheduler_manager.go Outdated Show resolved Hide resolved
@codecov-commenter
Copy link

Codecov Report

Merging #163 (25aa905) into main (494e15e) will increase coverage by 1.20%.
The diff coverage is 35.43%.

❗ Current head 25aa905 differs from pull request most recent head c63ea0f. Consider uploading reports for the commit c63ea0f to get more accurate results

@@            Coverage Diff             @@
##             main     #163      +/-   ##
==========================================
+ Coverage   39.07%   40.28%   +1.20%     
==========================================
  Files          33       34       +1     
  Lines        4217     4148      -69     
==========================================
+ Hits         1648     1671      +23     
+ Misses       2317     2252      -65     
+ Partials      252      225      -27     
Impacted Files Coverage Δ
server/cluster/metadata/cluster_metadata.go 0.00% <0.00%> (ø)
server/cluster/metadata/table_manager.go 0.00% <0.00%> (ø)
server/cluster/metadata/topology_manager.go 0.00% <0.00%> (ø)
server/cluster/metadata/types.go 0.00% <0.00%> (ø)
server/coordinator/procedure/manager_impl.go 0.00% <0.00%> (ø)
server/coordinator/procedure/util.go 0.00% <0.00%> (-38.89%) ⬇️
server/etcdutil/util.go 45.45% <0.00%> (-6.79%) ⬇️
server/coordinator/factory.go 4.51% <4.51%> (ø)
server/storage/types.go 63.54% <38.23%> (-7.01%) ⬇️
server/storage/storage_impl.go 62.30% <40.00%> (-1.21%) ⬇️
... and 14 more

... and 2 files with indirect coverage changes

Copy link
Member

@ShiKaiWi ShiKaiWi left a comment

Choose a reason for hiding this comment

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

Good Job! 👍👍

@ShiKaiWi ShiKaiWi merged commit 9608d4c into apache:main Apr 27, 2023
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.

3 participants