Task #10824 (new)
Opened 11 years ago
Last modified 9 years ago
RFE: NetworkChecker modifications — at Initial Version
| Reported by: | jamoore | Owned by: | sebastien.simard@… |
|---|---|---|---|
| Priority: | critical | Milestone: | 5.x |
| Component: | Insight | Version: | n.a. |
| Keywords: | n.a. | Cc: | jburel, cxallan, mtbcarroll, bpindelski, sylittlewood |
| Resources: | n.a. | Referenced By: | n.a. |
| References: | n.a. | Remaining Time: | n.a. |
| Sprint: | n.a. |
Description
See:
- https://github.com/openmicroscopy/openmicroscopy/pull/1117
- https://github.com/openmicroscopy/openmicroscopy/pull/1075
Now that isNetworkUp() is called on every remote invocation reducing blockages will become paramount.
Suggested improvements from a discussion with Chris Allan:
- Move from testing openmicroscopy.org.uk to testing the server/port that Ice is configured to use. This may require housing the NetworkChecker inside of the SecurityContext, though on shutdown it's unclear which one should be used.
- Set timeouts on all sockets. If using URL, set via URLConnection#setReadTimeout
Further improvements which could also be considered (perhaps as a second ticket):
- Check uses of getHostAddress for hanging (see https://github.com/openmicroscopy/openmicroscopy/pull/1075/files#L3R2384)
- Consider a reflection-based method which would use the improved Java 6 APIs if available.
Note: See
TracTickets for help on using
tickets.
You may also have a look at Agilo extensions to the ticket.
