diff options
author | Jon Marius Venstad <venstad@gmail.com> | 2021-02-07 11:49:09 +0100 |
---|---|---|
committer | Jon Marius Venstad <venstad@gmail.com> | 2021-02-07 11:49:09 +0100 |
commit | 68f19cb9619c1d171d8a9e5062eae5d985a6858a (patch) | |
tree | 7fc8b1124ea4d0324f7a670eecb220171d7245de /vespaclient-container-plugin/src/main | |
parent | 79709d8f64ef2a076c9e8dcbcd4dcf5f9eafabbf (diff) |
Bound time chunk to 5s less than request timeout
Diffstat (limited to 'vespaclient-container-plugin/src/main')
-rw-r--r-- | vespaclient-container-plugin/src/main/java/com/yahoo/document/restapi/resource/DocumentV1ApiHandler.java | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/vespaclient-container-plugin/src/main/java/com/yahoo/document/restapi/resource/DocumentV1ApiHandler.java b/vespaclient-container-plugin/src/main/java/com/yahoo/document/restapi/resource/DocumentV1ApiHandler.java index 65744815ecf..9b198ce8df9 100644 --- a/vespaclient-container-plugin/src/main/java/com/yahoo/document/restapi/resource/DocumentV1ApiHandler.java +++ b/vespaclient-container-plugin/src/main/java/com/yahoo/document/restapi/resource/DocumentV1ApiHandler.java @@ -948,7 +948,8 @@ public class DocumentV1ApiHandler extends AbstractRequestHandler { requireProperty(request, SELECTION); VisitorParameters parameters = parseCommonParameters(request, path, Optional.of(requireProperty(request, CLUSTER))); parameters.setThrottlePolicy(new DynamicThrottlePolicy().setMinWindowSize(1).setWindowSizeIncrement(1)); - parameters.setSessionTimeoutMs(Math.max(1, getProperty(request, TIME_CHUNK, timeoutMillisParser).orElse(60_000L))); + long timeChunk = getProperty(request, TIME_CHUNK, timeoutMillisParser).orElse(60_000L); + parameters.setSessionTimeoutMs(Math.max(1, Math.min(timeChunk, request.getTimeout(TimeUnit.MILLISECONDS) - 5000L))); return parameters; } |