Skip to content
This repository has been archived by the owner on Dec 17, 2018. It is now read-only.

Add Randomized AppendEntriesReply test #24

Closed
allengeorge opened this issue Dec 12, 2013 · 1 comment
Closed

Add Randomized AppendEntriesReply test #24

allengeorge opened this issue Dec 12, 2013 · 1 comment
Assignees
Milestone

Comments

@allengeorge
Copy link
Owner

SImilar to the AppendEntries test. It would be useful to come up with a randomized AppendEntriesReply test that exercises:

  1. Out of order replies.
  2. A mix of nack and ack replies.
  3. Varying numbers of added entries (to ensure that the nextIndex doesn't shrink).

...

@allengeorge
Copy link
Owner Author

The major thing missing from this implementation is a checking how onAppendEntriesReply can handle a mix of NACK and ACK replies. This is equivalent to putting the algorithm in either the PREFIX_SEARCH or APPLYING phase and then bombarding it with random messages. I'm not quite sure if there's a lot of value in that, although I'm open to revisiting this.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant