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
