eclipse-ee4j / glassfish-shoal

Shoal
Other
5 stars 9 forks source link

HealthMonitor.isConnected : for loop only looks at 1 network interface #71

Closed glassfishrobot closed 14 years ago

glassfishrobot commented 16 years ago

The power outage related code in the IndoubtPeerDetector thread's isConnected() method iterates over only 1 Future task and returns false if the future task is not done. The "return false" statement needs to be inside the catch block for InterruptedException. Otherwise the following 2 lines of code will never get executed in the case where the future task is not complete:

fine("Peer Machine for " + entry.adv.getName() + " is down!"); return false;

Environment

Operating System: All Platform: Macintosh

Affected Versions

[current]

glassfishrobot commented 6 years ago
glassfishrobot commented 16 years ago

@glassfishrobot Commented Reported by sheetalv

glassfishrobot commented 16 years ago

@glassfishrobot Commented sheetalv said: issue fixed in the trunk.

https://shoal.dev.java.net/servlets/ReadMsg?list=cvs&msgNo=636

glassfishrobot commented 16 years ago

@glassfishrobot Commented Was assigned to sheetalv

glassfishrobot commented 7 years ago

@glassfishrobot Commented This issue was imported from java.net JIRA SHOAL-71

glassfishrobot commented 14 years ago

@glassfishrobot Commented Marked as fixed on Wednesday, June 23rd 2010, 4:11:06 am