Skip to content

Dev 5.4.5 #143

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 7 commits into from
Oct 15, 2020
Merged

Dev 5.4.5 #143

merged 7 commits into from
Oct 15, 2020

Conversation

dzdx
Copy link
Member

@dzdx dzdx commented Oct 12, 2020 •
edited
Loading

  1. upgrade jraft to 1.3.5.Alpha1
  2. split meta thread pool to improve stability
  3. bugfix: repub will remove other publishers in same connectID
  4. session default heartbeat interval in properties
  5. add admin api to drop connections in session

@dzdx dzdx requested a review from nocvalight October 12, 2020 05:56
@codecov-io
Copy link

Codecov Report

Merging #143 into master will decrease coverage by 0.90%.
The diff coverage is 42.21%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #143      +/-   ##
============================================
- Coverage     62.93%   62.02%   -0.91%     
  Complexity       44       44              
============================================
  Files           429      437       +8     
  Lines         15502    15801     +299     
  Branches       1490     1502      +12     
============================================
+ Hits           9756     9801      +45     
- Misses         4735     4989     +254     
  Partials       1011     1011
Impacted Files Coverage Δ Complexity Δ
.../model/metaserver/ConfigureLoadbalanceRequest.java 0.00% <0.00%> (ø) 0.00 <0.00> (?)
...model/metaserver/GetLoadbalanceMetricsRequest.java 0.00% <0.00%> (ø) 0.00 <0.00> (?)
...ry/common/model/metaserver/LoadbalanceMetrics.java 0.00% <0.00%> (ø) 0.00 <0.00> (?)
...server/data/change/event/DataChangeEventQueue.java 89.02% <ø> (-0.14%) 0.00 <0.00> (ø)
...gistry/server/meta/bootstrap/MetaServerConfig.java 100.00% <ø> (ø) 0.00 <0.00> (ø)
.../server/meta/node/impl/SessionNodeServiceImpl.java 33.07% <0.00%> (-23.69%) 0.00 <0.00> (ø)
...istry/server/meta/registry/MetaServerRegistry.java 61.11% <0.00%> (-14.65%) 0.00 <0.00> (ø)
...eta/remoting/connection/DataConnectionHandler.java 66.66% <ø> (ø) 0.00 <0.00> (ø)
...eta/remoting/connection/MetaConnectionHandler.java 15.00% <ø> (ø) 0.00 <0.00> (ø)
.../remoting/connection/SessionConnectionHandler.java 70.58% <ø> (ø) 0.00 <0.00> (ø)
... and 68 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 505e36b...070699e. Read the comment docs.

@dzdx dzdx requested a review from quhongwei October 12, 2020 07:18
nocvalight
nocvalight approved these changes Oct 12, 2020
View reviewed changes
Copy link
Member

@nocvalight nocvalight left a comment

Choose a reason for hiding this comment

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

LGTM

@dzdx dzdx merged commit cc2f105 into sofastack:master Oct 15, 2020
dzdx added a commit that referenced this pull request Dec 13, 2021
@dzdx
Dev 5.4.5 (#143)
fd35670 
* upgrade version to 5.4.5

* bugfix: repub will remove other publishers in same connectID

* split meta thread pool to improve stability

* configurable electiontimeout

* add admin api to implement session connections loadbalance

* session heartbeat timeout

* upgrade jraft to 1.3.5.Alpha1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@quhongwei quhongwei quhongwei approved these changes

@nocvalight nocvalight nocvalight approved these changes

Assignees
No one assigned
Labels
cla:yes size/XXL
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@dzdx @codecov-io @quhongwei @nocvalight