Sort in descending order

goraft / raft

UNMAINTAINED: A Go implementation of the Raft distributed consensus protocol.

MIT License

2.43k stars 480 forks source link

Closed xiang90 closed 11 years ago

philips commented 11 years ago

LGTM :)

philips commented 11 years ago

This fixes #118 reported by @kellabyte

benbjohnson commented 11 years ago

@xiangli-cmu Can you change this to use sort.Reverse() instead? I think making uint64Slice sort in reverse order is going to bite us down the road.

philips commented 11 years ago

@benbjohnson Good idea. It was super confusing reading: Less returns i > j

xiang90 commented 11 years ago

@benbjohnson will do that.

benbjohnson commented 11 years ago

:+1: