Task #10824 (new)
Opened 11 years ago
Last modified 9 years ago
RFE: NetworkChecker modifications — at Version 3
| 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 (last modified by jamoore)
See:
- https://github.com/openmicroscopy/openmicroscopy/issues/821 (J-M)
- https://github.com/openmicroscopy/openmicroscopy/pull/1070 (Zeb)
- https://github.com/openmicroscopy/openmicroscopy/pull/1075 (J-M)
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.
Change History (3)
comment:1 Changed 11 years ago by jamoore
comment:2 Changed 11 years ago by jburel
Unfortunately it has to be made on every method call at the moment
Any Java 6 feature cannot be considered on 4.4.x since the minimum required is 1.5
comment:3 Changed 11 years ago by jamoore
- Description modified (diff)
Note: See
TracTickets for help on using
tickets.
You may also have a look at Agilo extensions to the ticket.
See: https://www.openmicroscopy.org/qa2/qa/feedback/6622