goldcollection
/
muwire
mirror of https://github.com/zlatinb/muwire
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

make sure hosts with successful attempts can be advertised

pull/53/head
Zlatin Balevsky 2020-10-21 06:10:14 +01:00
parent 38bf4f2ea2
commit efae38a400
No known key found for this signature in database
GPG Key ID: A72832072D525E41
2 changed files with 13 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
core/src/main/groovy/com/muwire/core/hostcache/H2HostCache.groovy
View File

@ -43,11 +43,11 @@ class H2HostCache extends HostCache {
// overwrite MC with optimistic values
if (fromHostcache) {
sql.execute("delete from HOST_ATTEMPTS where DESTINATION=${d.toBase64()}")
profiles.put(d, new HostMCProfile())
profiles.put(d, new HostMCProfile(false))
}
if (uniqueHosts.add(d)) {
allHosts.add(d)
profiles.put(d, new HostMCProfile())
profiles.put(d, new HostMCProfile(false))
}
}
@ -56,9 +56,12 @@ class H2HostCache extends HostCache {
log.fine("onConnection ${d.toBase32()} status $status")
if (status == ConnectionAttemptStatus.SUCCESSFUL && uniqueHosts.add(d)) {
allHosts.add(d)
profiles.put(d, new HostMCProfile())
if (status == ConnectionAttemptStatus.SUCCESSFUL) {
if (uniqueHosts.add(d)) {
allHosts.add(d)
profiles.put(d, new HostMCProfile(true))
} else
profiles.get(d).successfulAttempt = true
}
// record into db
@ -285,7 +288,7 @@ class H2HostCache extends HostCache {
Destination dest = new Destination(it.DESTINATION)
if (uniqueHosts.add(dest)) {
def fromDB = sql.firstRow("select * from HOST_PROFILES where DESTINATION=${dest.toBase64()}")
def profile = new HostMCProfile()
def profile = new HostMCProfile(true)
if (fromDB != null)
profile = new HostMCProfile(fromDB)
profiles.put(dest, profile)

7
core/src/main/groovy/com/muwire/core/hostcache/HostMCProfile.groovy
View File

@ -38,12 +38,13 @@ class HostMCProfile {
ConnectionAttemptStatus state = ConnectionAttemptStatus.SUCCESSFUL
final boolean hasHistory
boolean successfulAttempt
/**
* constructs an "optimistic" predictor for newly discovered hosts.
*/
HostMCProfile() {
hasHistory = false
HostMCProfile(boolean hasHistory) {
this.hasHistory = hasHistory
S = new Probability[3]
R = new Probability[3]
F = new Probability[3]
@ -159,7 +160,7 @@ class HostMCProfile {
* @return if the host should be advertised in pongs
*/
boolean shouldAdvertise() {
hasHistory && nextState() != ConnectionAttemptStatus.FAILED
(hasHistory || successfulAttempt) && nextState() != ConnectionAttemptStatus.FAILED
}
/**