spmallette commented on code in PR #3508: URL: https://github.com/apache/tinkerpop/pull/3508#discussion_r3539395885
########## docs/src/dev/provider/index.asciidoc: ########## @@ -1430,8 +1430,8 @@ rejects it with HTTP 400. This prevents cross-graph operations within a single t ==== Transaction Timeout and Idle Reclamation -A transaction can be bounded at three independent scopes, each disabled with `0`. The `evaluationTimeout` bounds a -single operation. The `idleTransactionTimeout` (default 60000ms) bounds the idle gaps between operations: how long a +A transaction can be bounded at three independent scopes, each disabled with `0`. The `timeoutMs` bounds a single +operation. The `idleTransactionTimeout` (default 60000ms) bounds the idle gaps between operations: how long a Review Comment: i was wondering what was going to happen with `idleTransactionTimeout` (and other timeouts that may still exist) in this PR. Did we now create another inconsistency by moving to the `Ms` suffix? Should all "timeouts" have the suffix so that it all looks the same? -- 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]
