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

Refactoring Cluster support in suuchi #68

Merged
merged 7 commits into from
Jun 2, 2017
Merged

Conversation

ashwanthkumar
Copy link
Owner

Fixing both the atomix + scalecube implementations. We don't have cluster integrated into Server yet, but this is baby steps towards it.

- Added typesafe config dependency on suuchi-core
- Fixing Atomix cluster implementation + Added ClusterProvider for Atomix cluster
- Compiling the whole project still on 1.7 but Atomix alone to 1.8
- Improving the README on Atomix implementation
- ClusterProvider#createCluster now takes typesafe Config as a param so we can pass around additional settings to cluster implementations.
@codecov-io
Copy link

codecov-io commented Jun 2, 2017

Codecov Report

Merging #68 into master will increase coverage by 0.51%.
The diff coverage is 84.61%.

@@             Coverage Diff              @@
##             master      #68      +/-   ##
============================================
+ Coverage     73.65%   74.17%   +0.51%     
- Complexity      115      121       +6     
============================================
  Files            33       34       +1     
  Lines           577      604      +27     
  Branches         74       78       +4     
============================================
+ Hits            425      448      +23     
- Misses           90       91       +1     
- Partials         62       65       +3

@ashwanthkumar ashwanthkumar merged commit 0d46a21 into master Jun 2, 2017
@ashwanthkumar ashwanthkumar deleted the cluster-integration branch June 2, 2017 06:09
@ashwanthkumar ashwanthkumar added this to the 0.2 milestone Jun 3, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants