partisan icon indicating copy to clipboard operation
partisan copied to clipboard

update_members should only accept a list of maps

Open tsloughter opened this issue 7 years ago • 1 comments

The default peer service manager currently supports the nodes list as both a list of atoms and a list of maps. This results in some confusing hacks and logic to handle.

Requiring the list to be the map representation of a node would clear it up and allow to simply use sets to find the difference and intersection of the current membership and the updated membership.

tsloughter avatar Nov 11 '17 17:11 tsloughter

I will take care of this soon.

cmeiklejohn avatar Nov 11 '17 18:11 cmeiklejohn