agorise/graphenej
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Returning latency as Long.MAX_VALUE instead of zero to non-initialized FullNode instances

Browse source
This commit is contained in:
Nelson R. Perez 2018-10-18 17:57:27 -05:00
parent 63eebf11c4
commit d37d35e0c6
2 changed files with 8 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
graphenej/src/main/java/cy/agorise/graphenej/network/FullNode.java
View file

@ -65,10 +65,16 @@ public class FullNode implements Comparable {
/** /**
* *
* @return The latest latency average value * @return The latest latency average value. If no measurement has been taken yet, the
* maximum allows value of a long primitive, or 2<sup>63</sup>-1 will be returned.
*/ */
public double getLatencyValue() { public double getLatencyValue() {
return mLatency.getAverage(); double average = mLatency.getAverage();
if(average == 0){
return Long.MAX_VALUE;
}else{
return average;
}
} }
public boolean isConnected() { public boolean isConnected() {

2
graphenej/src/main/java/cy/agorise/graphenej/network/NodeLatencyVerifier.java
View file

@ -35,8 +35,6 @@ public class NodeLatencyVerifier {
private HashMap<HttpUrl, FullNode> nodeURLMap = new HashMap<>(); private HashMap<HttpUrl, FullNode> nodeURLMap = new HashMap<>();
// private WebSocket webSocket;
// Map used to store the first timestamp required for a RTT (Round Trip Time) measurement. // Map used to store the first timestamp required for a RTT (Round Trip Time) measurement.
// If: // If:
// RTT = t2 - t1 // RTT = t2 - t1