Skip to content

Conversation

@Elia-Renzoni
Copy link

I closed the previous PR because it was minimal, and I wanted to do something more complete.
In this PR, I’ve added further changes and improvements compared to the previous version.

Let me know what you think!

@JaylinYu
Copy link
Member

I would propose a new nano-raft branch for exploring the clustering feature. this is not sth could be chopped into pieces as small PR.

@Elia-Renzoni
Copy link
Author

Hi @JaylinYu
the clustering feature will need to be implemented in NanoMQ, and I’ll follow your suggestion to work locally and only open a PR once everything is complete.

The idea behind this PR was to already have the necessary raft-related data structures merged into the NanoNNG submodule, so that part would be in place.
However, I can definitely work on a single PR that includes everything instead.

@JaylinYu
Copy link
Member

I am only saying we can have an independent branch for this.
you can push your PR to Elia/raft now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants