andreachild commented on code in PR #3159:
URL: https://github.com/apache/tinkerpop/pull/3159#discussion_r2213786369
##########
gremlin-server/src/main/java/org/apache/tinkerpop/gremlin/server/Settings.java:
##########
@@ -210,21 +210,29 @@ public Settings() {
* created {@link Session} will queue separately given that setting per
session.
* <p/>
* By default this value is set to 8192.
+ *
+ * @deprecated As of release 3.8.0, not replaced.
*/
+ @Deprecated
Review Comment:
I'm in favour of not deprecating it as its usage in
`ServerGremlinExecutor.java` has not changed from 3.8 to master. What has
changed is only its relevance to sessions, hence the removal of the javadoc
section below:
> This value should be taken in account with the {@link
#maxSessionTaskQueueSize} which is related in some respects. A request that
starts a new {@link Session} is handled by this queue, but additional requests
to a created {@link Session} will queue separately given that setting per
session.
--
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]
