Fixed
Details
Assignee
Seth LegerSeth LegerReporter
Alejandro GalueAlejandro GalueLabels
Components
Fix versions
Priority
Major
Details
Details
Assignee
Seth Leger
Seth LegerReporter
Alejandro Galue
Alejandro GalueLabels
Components
Fix versions
Priority
PagerDuty
PagerDuty
PagerDuty
Created June 10, 2013 at 10:50 AM
Updated April 26, 2016 at 1:32 PM
Resolved April 26, 2016 at 1:32 PM
We usually expect to retry some operation if we receive a timeout from the request.
If we use the DefaultRetryHandler from the HttpClient library that is not going to happen because that handler is not catching ConnectTimeoutException and/or SocketTimeoutException. So, when any of those exception happen, the operation will be aborted and the HttpClient is not going to "try again". If we want to "try again" we must implement a custom RetryHandler.
For this reason only the "sequence-retry" is the only operation related with retries that is going to work on the PSM (or any other piece of code in OpenNMS that is using the DefaultRetryHandler with the HttpClient).
If we have a definition with only one page that is a correct workaround, but if we have several pages, it is too expensive to use a sequence-retry instead of a simple retry, because the sequence-retry is going to start over with the whole test (i.e., test all the pages again), and the single retry is at page level (i.e., on each HttpGet/HttpPost request).