Skip to content
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.

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

fix duplicate connectId && snapshot backup #135

Merged
merged 5 commits into from
Aug 23, 2020
Merged

Conversation

dzdx
Copy link
Member

@dzdx dzdx commented Aug 23, 2020

  • duplicate connectId fix
  • snapshot backup
  • upgrade version 5.4.4
  • upgrade rocksdbjni

@codecov-commenter
Copy link

codecov-commenter commented Aug 23, 2020

Codecov Report

Merging #135 into master will increase coverage by 0.21%.
The diff coverage is 50.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #135      +/-   ##
============================================
+ Coverage     62.03%   62.25%   +0.21%     
  Complexity       44       44              
============================================
  Files           429      429              
  Lines         15500    15502       +2     
  Branches       1490     1490              
============================================
+ Hits           9616     9650      +34     
+ Misses         4876     4843      -33     
- Partials       1008     1009       +1     
Impacted Files Coverage Δ Complexity Δ
...istry/server/session/registry/SessionRegistry.java 76.00% <0.00%> (-0.44%) 0.00 <0.00> (ø)
...server/data/renew/LocalDataServerCleanHandler.java 28.12% <50.00%> (+1.56%) 0.00 <0.00> (ø)
...er/data/bootstrap/DataServerBeanConfiguration.java 100.00% <100.00%> (ø) 0.00 <0.00> (ø)
...ion/scheduler/task/ProvideDataChangeFetchTask.java 52.54% <100.00%> (ø) 0.00 <0.00> (ø)
...ipay/sofa/registry/jraft/bootstrap/RaftServer.java 73.91% <0.00%> (ø) 0.00% <0.00%> (ø%)
...a/registry/server/meta/remoting/RaftExchanger.java 45.93% <0.00%> (ø) 0.00% <0.00%> (ø%)
...a/registry/server/meta/store/DataStoreService.java 66.54% <0.00%> (+0.37%) 0.00% <0.00%> (ø%)
...rver/session/node/service/DataNodeServiceImpl.java 67.77% <0.00%> (+0.55%) 0.00% <0.00%> (ø%)
...a/registry/server/data/datasync/sync/Acceptor.java 70.31% <0.00%> (+0.78%) 0.00% <0.00%> (ø%)
...y/sofa/registry/task/batcher/AcceptorExecutor.java 72.72% <0.00%> (+0.82%) 0.00% <0.00%> (ø%)
... and 7 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 3a2034a...c3b4e09. Read the comment docs.

@dzdx dzdx requested a review from nocvalight August 23, 2020 05:34
@dzdx dzdx merged commit 505e36b into sofastack:master Aug 23, 2020
nocvalight pushed a commit to nocvalight/sofa-registry that referenced this pull request Aug 23, 2020
* upgrade version to 5.4.4-SNAPSHOT

* update version 5.4.4

* deplicate conn id

* fix snapshot backup

* watcher connectid four tuple
dzdx added a commit that referenced this pull request Dec 13, 2021
* upgrade version to 5.4.4-SNAPSHOT

* update version 5.4.4

* deplicate conn id

* fix snapshot backup

* watcher connectid four tuple
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working cla:yes size/M
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants