spmallette commented on code in PR #3159:
URL: https://github.com/apache/tinkerpop/pull/3159#discussion_r2213764129
##########
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:
hmm, not sure...looks like its been repurposed for HTTP. not sure i've ever
had that before. in a way it's deprecated in the context of how it is being
used here so in that way it makes sense, but the fact that as an API element
it's not going to be removed makes it not deprecated.
i guess i'd leave it deprecated but if you/others feel it should not be, i
can change it.
--
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]
