spmallette commented on code in PR #3508: URL: https://github.com/apache/tinkerpop/pull/3508#discussion_r3539437891
########## docs/src/upgrade/release-4.x.x.asciidoc: ########## @@ -106,6 +106,29 @@ These change runtime behavior on upgrade even if you do not change your configur See: link:https://lists.apache.org/thread/yqtr2wnb1kq2pqqq4002cz511q5o0bkg[[DISCUSS] Standardizing GLV connection options in TinkerPop 4]. +==== Renaming `evaluationTimeout` to `timeoutMs` + +The per-request execution timeout is now referred to by a single name, `timeoutMs`, everywhere. `timeoutMs` is the +maximum time in milliseconds that a request is allowed to execute on the server before it times out; it can be +configured server-wide and overridden on a per-request basis. Previously the same concept was called +`evaluationTimeout` in the server configuration, the `with()` script token, and several driver APIs, while the wire +protocol already used `timeoutMs` — collapsing to one name removes that inconsistency. Review Comment: — -- 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]
