divijvaidya commented on code in PR #1833:
URL: https://github.com/apache/tinkerpop/pull/1833#discussion_r1003199087
##########
gremlin-driver/src/main/java/org/apache/tinkerpop/gremlin/driver/Connection.java:
##########
@@ -59,14 +59,15 @@ final class Connection {
public static final int MAX_IN_PROCESS = 4;
public static final int MIN_IN_PROCESS = 1;
- public static final int MAX_WAIT_FOR_CONNECTION = 16000;
+ public static final int MAX_WAIT_FOR_CONNECTION = 25000;
Review Comment:
You are right, it's not an interface / contract change but if we release
this as a patch change, a customer may have to change their application code to
consume the patch version.
As an example, let's say that the customer has set exponential backoff/retry
for connection setup based on the current default values but now we are
changing the default values and their tuning for retries would break. That is
the reason, I would like to add this to 3.7.x.
@spmallette thoughts?
--
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]
