Actually one other consistency change I'd like to make. Since the GLVs all use the suffix "Millis", I'm proposing we change the field name in the HTTP API and the server to the "Millis" as well. so it would be "timeoutMillis" instead of "timeoutMs". All server settings above would have the "Millis" suffix applied to them.
On Fri, Jul 10, 2026 at 12:45 PM Ken Hu <[email protected]> wrote: > Hi All, > > While working on https://github.com/apache/tinkerpop/pull/3508 which > renames evaluationTimeout to timeoutMs in the server to match the HTTP API > field, it was noted that this creates an inconsistency with all the other > options in the server. To resolve this, I'm suggesting we add an "Ms" > suffix to all time related settings in 4.x. This would include: > > idleConnectionTimeout > keepAliveInterval > idleTransactionTimeout > perGraphCloseTimeout > maxTransactionLifetime > SslSettings.refreshInterval > IntervalMetrics.interval > > e.g. it would go from idleConnectionTimeout to idleConnectionTimeoutMs to > keep it consistent. > > Any concerns around this? > > Thanks, > Ken >
