GetLeaderRaftConnectString returns consistent result

[oliveagle]

return a consistant leader raft connection string when called on all living nodes include leader.

[jvshahid]

@oliveagle can you explain or give an example of why the old version is inconsistent vs the new one

[oliveagle]

on leader node. the old version api will return empty string ""
on follower node. the old api will return leader raft string.

IMHO, isLeader or not should be hold in another variable. otherwise , u have to explain this behavior in doc or some other place.

"isLeader": only return True or False. indicator the server is leader or not.
"leaderRaftConnectString": only return leader raft connection string. even the node is the leader.

by doing this way. the variable is totally self-explained. If the cluster has one leader, then there is only one value will be returned cross all nodes.

well, IsUp is more interesting. when called on A node, it shows B node is Up. A nodes is down.
when called on B node, it shows A node is UP, B node is down. isUp variable is private but IsUp function is public.