feat(core): cluster role automatic management by zyxxoo · Pull Request #1943 · apache/hugegraph
Navigation Menu
{{ message }}
- Notifications You must be signed in to change notification settings
- Fork 583
Conversation
Copy link
Codecov Report
Merging #1943 (67a5f93) into master (7274a5f) will increase coverage by
0.05%.
The diff coverage is69.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
changed the title
#1869 cluster role automatic management
feat(core): cluster role automatic management
imbajin reviewed Aug 15, 2022
hugegraph-core/src/main/java/com/baidu/hugegraph/election/MetaData.java
Outdated
Show resolved
Hide resolved
hugegraph-core/src/main/java/com/baidu/hugegraph/election/MetaData.java Outdated Show resolved Hide resolved
| public class MetaData { | ||
|
|
||
| String node; | ||
| long count; |
Copy link
Member
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
Contributor Author
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe int is enough
hugegraph-core/src/main/java/com/baidu/hugegraph/election/MetaDataAdapter.java
Outdated
Show resolved
Hide resolved
hugegraph-core/src/main/java/com/baidu/hugegraph/election/MetaDataAdapter.java Outdated Show resolved Hide resolved
zyxxoo
force-pushed
the
zy_dev
branch
from
6ba0bb3 to
54dac30
Compare
zyxxoo
force-pushed
the
zy_dev
branch
from
54dac30 to
99690d8
Compare
javeme reviewed Aug 16, 2022
hugegraph-core/src/main/java/com/baidu/hugegraph/election/RoleElectionStateMachineImpl.java
Outdated
Show resolved
Hide resolved
hugegraph-core/src/main/java/com/baidu/hugegraph/election/RoleElectionStateMachineImpl.java Outdated Show resolved Hide resolved
hugegraph-core/src/main/java/com/baidu/hugegraph/election/RoleElectionStateMachineImpl.java
Outdated
Show resolved
Hide resolved
hugegraph-core/src/main/java/com/baidu/hugegraph/election/RoleElectionStateMachineImpl.java Outdated Show resolved Hide resolved
hugegraph-core/src/main/java/com/baidu/hugegraph/election/RoleElectionStateMachineImpl.java
Outdated
Show resolved
Hide resolved
hugegraph-core/src/main/java/com/baidu/hugegraph/election/RoleElectionStateMachineImpl.java Outdated Show resolved Hide resolved
hugegraph-core/src/main/java/com/baidu/hugegraph/election/RoleElectionStateMachineImpl.java
Outdated
Show resolved
Hide resolved
hugegraph-core/src/main/java/com/baidu/hugegraph/election/RoleElectionStateMachineImpl.java Outdated Show resolved Hide resolved
hugegraph-core/src/main/java/com/baidu/hugegraph/election/RoleElectionStateMachineImpl.java
Outdated
Show resolved
Hide resolved
hugegraph-core/src/main/java/com/baidu/hugegraph/election/RoleElectionStateMachineImpl.java Outdated Show resolved Hide resolved
hugegraph-core/src/main/java/com/baidu/hugegraph/election/RoleElectionStateMachine.java
Outdated
Show resolved
Hide resolved
hugegraph-core/src/main/java/com/baidu/hugegraph/election/RoleElectionStateMachine.java Outdated Show resolved Hide resolved
hugegraph-core/src/main/java/com/baidu/hugegraph/election/RoleElectionStateMachineImpl.java
Outdated
Show resolved
Hide resolved
hugegraph-core/src/main/java/com/baidu/hugegraph/election/RoleElectionStateMachineImpl.java Outdated Show resolved Hide resolved
hugegraph-core/src/main/java/com/baidu/hugegraph/election/RoleElectionStateMachineImpl.java
Outdated
Show resolved
Hide resolved
hugegraph-core/src/main/java/com/baidu/hugegraph/election/RoleElectionStateMachineImpl.java Outdated Show resolved Hide resolved
hugegraph-core/src/main/java/com/baidu/hugegraph/election/MetaDataAdapter.java
Outdated
Show resolved
Hide resolved
hugegraph-core/src/main/java/com/baidu/hugegraph/election/MetaDataAdapter.java Outdated Show resolved Hide resolved
hugegraph-core/src/main/java/com/baidu/hugegraph/election/MetaDataAdapter.java
Outdated
Show resolved
Hide resolved
hugegraph-core/src/main/java/com/baidu/hugegraph/election/MetaDataAdapter.java Outdated Show resolved Hide resolved
zyxxoo
force-pushed
the
zy_dev
branch
from
ca598ce to
d366667
Compare
zyxxoo
force-pushed
the
zy_dev
branch
5 times, most recently
from
05932e0 to
d31b13d
Compare
zyxxoo
force-pushed
the
zy_dev
branch
from
d31b13d to
f342b47
Compare
javeme reviewed Aug 17, 2022
hugegraph-core/src/main/java/com/baidu/hugegraph/election/RoleElectionStateMachineImpl.java
Outdated
Show resolved
Hide resolved
hugegraph-core/src/main/java/com/baidu/hugegraph/election/RoleElectionStateMachineImpl.java Outdated Show resolved Hide resolved
hugegraph-core/src/main/java/com/baidu/hugegraph/election/RoleStataDataAdapter.java
Outdated
Show resolved
Hide resolved
hugegraph-core/src/main/java/com/baidu/hugegraph/election/RoleStataDataAdapter.java Outdated Show resolved Hide resolved
hugegraph-core/src/main/java/com/baidu/hugegraph/election/RoleStateData.java
Outdated
Show resolved
Hide resolved
hugegraph-core/src/main/java/com/baidu/hugegraph/election/RoleStateData.java Outdated Show resolved Hide resolved
hugegraph-core/src/main/java/com/baidu/hugegraph/election/StateMachineCallback.java
Outdated
Show resolved
Hide resolved
hugegraph-core/src/main/java/com/baidu/hugegraph/election/StateMachineCallback.java Outdated Show resolved Hide resolved
hugegraph-test/src/main/java/com/baidu/hugegraph/core/RoleElectionStateMachineTest.java
Outdated
Show resolved
Hide resolved
hugegraph-test/src/main/java/com/baidu/hugegraph/core/RoleElectionStateMachineTest.java Outdated Show resolved Hide resolved
hugegraph-core/src/main/java/com/baidu/hugegraph/election/RoleElectionStateMachineImpl.java
Outdated
Show resolved
Hide resolved
hugegraph-core/src/main/java/com/baidu/hugegraph/election/RoleElectionStateMachineImpl.java Outdated Show resolved Hide resolved
hugegraph-core/src/main/java/com/baidu/hugegraph/election/RoleElectionStateMachineImpl.java
Outdated
Show resolved
Hide resolved
hugegraph-core/src/main/java/com/baidu/hugegraph/election/RoleElectionStateMachineImpl.java Outdated Show resolved Hide resolved
hugegraph-core/src/main/java/com/baidu/hugegraph/election/RoleStateData.java
Outdated
Show resolved
Hide resolved
hugegraph-core/src/main/java/com/baidu/hugegraph/election/RoleStateData.java Outdated Show resolved Hide resolved
hugegraph-test/src/main/java/com/baidu/hugegraph/core/RoleElectionStateMachineTest.java
Outdated
Show resolved
Hide resolved
hugegraph-test/src/main/java/com/baidu/hugegraph/core/RoleElectionStateMachineTest.java Outdated Show resolved Hide resolved
hugegraph-core/src/main/java/com/baidu/hugegraph/election/RoleElectionStateMachineImpl.java
Outdated
Show resolved
Hide resolved
hugegraph-core/src/main/java/com/baidu/hugegraph/election/RoleElectionStateMachineImpl.java Outdated Show resolved Hide resolved
zyxxoo
force-pushed
the
zy_dev
branch
3 times, most recently
from
c46ace1 to
b458556
Compare
zyxxoo
force-pushed
the
zy_dev
branch
from
b458556 to
f3c5ab6
Compare
javeme reviewed Aug 17, 2022
hugegraph-core/src/main/java/com/baidu/hugegraph/election/RoleElectionStateMachineImpl.java
Outdated
Show resolved
Hide resolved
hugegraph-core/src/main/java/com/baidu/hugegraph/election/RoleElectionStateMachineImpl.java Outdated Show resolved Hide resolved
hugegraph-core/src/main/java/com/baidu/hugegraph/election/RoleElectionStateMachineImpl.java
Outdated
Show resolved
Hide resolved
hugegraph-core/src/main/java/com/baidu/hugegraph/election/RoleElectionStateMachineImpl.java Outdated Show resolved Hide resolved
hugegraph-test/src/main/java/com/baidu/hugegraph/core/RoleElectionStateMachineTest.java
Outdated
Show resolved
Hide resolved
hugegraph-test/src/main/java/com/baidu/hugegraph/core/RoleElectionStateMachineTest.java Outdated Show resolved Hide resolved
hugegraph-core/src/main/java/com/baidu/hugegraph/election/RoleTypeDataAdapter.java
Show resolved
Hide resolved
hugegraph-core/src/main/java/com/baidu/hugegraph/election/RoleTypeDataAdapter.java Show resolved Hide resolved
zyxxoo added 2 commits
August 17, 2022 11:14xiaoleizi2016 pushed a commit to xiaoleizi2016/incubator-hugegraph that referenced this pull request
Aug 20, 2022xiaoleizi2016 pushed a commit to xiaoleizi2016/incubator-hugegraph that referenced this pull request
Aug 22, 2022javeme previously approved these changes Aug 23, 2022
hugegraph-test/src/main/java/com/baidu/hugegraph/core/RoleElectionStateMachineTest.java
Outdated
Show resolved
Hide resolved
hugegraph-test/src/main/java/com/baidu/hugegraph/core/RoleElectionStateMachineTest.java Outdated Show resolved Hide resolved
hugegraph-test/src/main/java/com/baidu/hugegraph/core/RoleElectionStateMachineTest.java
Outdated
Show resolved
Hide resolved
hugegraph-test/src/main/java/com/baidu/hugegraph/core/RoleElectionStateMachineTest.java Outdated Show resolved Hide resolved
hugegraph-test/src/main/java/com/baidu/hugegraph/core/RoleElectionStateMachineTest.java
Outdated
Show resolved
Hide resolved
hugegraph-test/src/main/java/com/baidu/hugegraph/core/RoleElectionStateMachineTest.java Outdated Show resolved Hide resolved
zyxxoo
dismissed
javeme’s stale review
via
67a5f93
javeme approved these changes Aug 30, 2022
xiaoleizi2016 pushed a commit to xiaoleizi2016/incubator-hugegraph that referenced this pull request
Sep 6, 2022xiaoleizi2016 added a commit to xiaoleizi2016/incubator-hugegraph that referenced this pull request
Sep 6, 2022imbajin approved these changes Sep 16, 2022
zyxxoo
merged commit
51d29ac
into
master
zyxxoo
deleted the
zy_dev
branch
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment