Skip to content

feat(core): cluster role automatic management#1943

Merged
zyxxoo merged 9 commits intomasterfrom
zy_dev
Sep 16, 2022
Merged

feat(core): cluster role automatic management#1943
zyxxoo merged 9 commits intomasterfrom
zy_dev

Conversation

@zyxxoo
Copy link
Copy Markdown
Contributor

@zyxxoo zyxxoo commented Aug 15, 2022

No description provided.

@codecov
Copy link
Copy Markdown

codecov bot commented Aug 15, 2022 •
edited
Loading

Codecov Report

Merging #1943 (67a5f93) into master (7274a5f) will increase coverage by 0.05%.
The diff coverage is 69.59%.

@@             Coverage Diff              @@
##             master    #1943      +/-   ##
============================================
+ Coverage     70.47%   70.53%   +0.05%     
  Complexity      724      724              
============================================
  Files           452      454       +2     
  Lines         38984    39138     +154     
  Branches       5554     5574      +20     
============================================
+ Hits          27473    27605     +132     
- Misses         8814     8824      +10     
- Partials       2697     2709      +12
Impacted Files Coverage Δ
...ava/com/baidu/hugegraph/election/RoleTypeData.java 62.50% <62.50%> (ø)
...gegraph/election/RoleElectionStateMachineImpl.java 70.96% <70.96%> (ø)
...va/com/baidu/hugegraph/util/collection/IntMap.java 72.80% <0.00%> (-1.21%) ⬇️
...aidu/hugegraph/backend/store/hbase/HbaseTable.java 76.36% <0.00%> (-0.61%) ⬇️
...va/com/baidu/hugegraph/backend/cache/RamCache.java 72.22% <0.00%> (-0.56%) ⬇️
...m/baidu/hugegraph/backend/tx/GraphTransaction.java 79.93% <0.00%> (-0.21%) ⬇️
...gegraph/backend/cache/CachedSchemaTransaction.java 90.65% <0.00%> (+0.13%) ⬆️
...hugegraph/backend/store/raft/rpc/RaftRequests.java 20.12% <0.00%> (+0.22%) ⬆️
...om/baidu/hugegraph/backend/store/BackendTable.java 85.11% <0.00%> (+0.59%) ⬆️
...ugegraph/backend/store/raft/StoreStateMachine.java 65.76% <0.00%> (+0.90%) ⬆️
... and 8 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@imbajin imbajin changed the title #1869 cluster role automatic management feat(core): cluster role automatic management Aug 15, 2022
imbajin
imbajin reviewed Aug 15, 2022
View reviewed changes
hugegraph-core/src/main/java/com/baidu/hugegraph/election/MetaData.java Outdated
public class MetaData {

String node;
long count;
Copy link
Copy Markdown
Member

@imbajin imbajin Aug 15, 2022

Choose a reason for hiding this comment

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

do we need long?

Copy link
Copy Markdown
Contributor Author

@zyxxoo zyxxoo Aug 16, 2022

Choose a reason for hiding this comment

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

maybe int is enough

javeme
javeme reviewed Aug 16, 2022
View reviewed changes
@zyxxoo zyxxoo force-pushed the zy_dev branch 5 times, most recently from 05932e0 to d31b13d Compare August 16, 2022 09:43
javeme
javeme reviewed Aug 17, 2022
View reviewed changes
@zyxxoo zyxxoo force-pushed the zy_dev branch 3 times, most recently from c46ace1 to b458556 Compare August 17, 2022 01:52
javeme
javeme reviewed Aug 17, 2022
View reviewed changes
xiaoleizi2016 pushed a commit to xiaoleizi2016/incubator-hugegraph that referenced this pull request Aug 20, 2022
fix: modify abnormal logs(apache#1943)
682ea82
xiaoleizi2016 pushed a commit to xiaoleizi2016/incubator-hugegraph that referenced this pull request Aug 22, 2022
fix: modify abnormal logs(apache#1943)
bcb7bb4
javeme
javeme previously approved these changes Aug 23, 2022
View reviewed changes
xiaoleizi2016 pushed a commit to xiaoleizi2016/incubator-hugegraph that referenced this pull request Sep 6, 2022
@xiaoleizi2016
fix: modify abnormal logs(apache#1943)
9076d77
xiaoleizi2016 added a commit to xiaoleizi2016/incubator-hugegraph that referenced this pull request Sep 6, 2022
@xiaoleizi2016
fix: modify abnormal logs(apache#1943)
d7afcbb
@zyxxoo zyxxoo merged commit 51d29ac into master Sep 16, 2022
@zyxxoo zyxxoo deleted the zy_dev branch September 16, 2022 02:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@javeme javeme javeme approved these changes

@imbajin imbajin imbajin approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@zyxxoo @javeme @imbajin