spmallette commented on code in PR #3508:
URL: https://github.com/apache/tinkerpop/pull/3508#discussion_r3539421039


##########
docs/src/reference/gremlin-applications.asciidoc:
##########
@@ -985,6 +984,7 @@ The following table describes the various YAML 
configuration options that Gremli
 |strictTransactionManagement |Set to `true` to require `aliases` to be 
submitted on every requests, where the `aliases` become the scope of 
transaction management. |false
 |threadPoolBoss |The number of threads available to Gremlin Server for 
accepting connections. Should always be set to `1`. |1
 |threadPoolWorker |The number of threads available to Gremlin Server for 
processing non-blocking reads and writes. |1
+|timeoutMs |The maximum amount of time in milliseconds that a request is 
allowed to execute on the server before it times out. This is the server-wide 
default and may be overridden on a per-request basis with the `timeoutMs` 
request argument. This feature can be turned off by setting the value to `0`. 
|30000

Review Comment:
   Maybe this this is documented elsewhere, but a question that comes up often 
is "when does the timeout apply?" does it start when it hits the server (an 
queues)? or when it starts to be evaluated for execution as it gets picked up 
off the queue (on a busy server could add delays)? Not sure of the answer to 
that on HTTP - is that defined by the server semantics? 
   
   Perhaps a different issue, but I'd like to not lose that concept if not 
clearly stated.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to