You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tinkerpop.apache.org by sp...@apache.org on 2021/04/30 10:25:23 UTC

[tinkerpop] branch travis-fix updated (ee616c4 -> e69a17d)

This is an automated email from the ASF dual-hosted git repository.

spmallette pushed a change to branch travis-fix
in repository https://gitbox.apache.org/repos/asf/tinkerpop.git.


 discard ee616c4  Turn on start/end logging for individual tests.
     add 9c9c01d  Decided to revert back to having a dedicated thread for host initialization and keeping the scheduler for reconnect.
     add 7aeed79  Merge branch '3.4-dev'
     add e6bebd4  Turn on start/end logging for individual tests.
     add 9acaaed  Reverted to older version of surefire for performance reasons
     add fdad506  Docs were failing with this escape.
     add 8ff5abb  Merge branch '3.4-dev'
     add f5614be  Added some links jira in upgrade docs CTR
     add c2f2bc2  Fixed minor nits in the Upgrade docs for 3.5.0 CTR
     add b9ece1e  Fixed capitalization CTR
     add e71a7a9  Replaced blocking calls on the test client with timeouts.
     new e69a17d  Hardcode threadpool size to 2.

This update added new revisions after undoing existing revisions.
That is to say, some revisions that were in the old version of the
branch are not in the new version.  This situation occurs
when a user --force pushes a change and generates a repository
containing something like this:

 * -- * -- B -- O -- O -- O   (ee616c4)
            \
             N -- N -- N   refs/heads/travis-fix (e69a17d)

You should already have received notification emails for all of the O
revisions, and so the following emails describe only the N revisions
from the common base, B.

Any revisions marked "omit" are not gone; other references still
refer to them.  Any revisions marked "discard" are gone forever.

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 CHANGELOG.asciidoc                                 |  2 +-
 docs/src/upgrade/release-3.5.x.asciidoc            | 61 ++++++++++++----------
 .../apache/tinkerpop/gremlin/driver/Client.java    | 14 ++++-
 .../apache/tinkerpop/gremlin/driver/Cluster.java   |  4 +-
 .../gremlin/driver/simple/AbstractClient.java      | 10 ++--
 .../gremlin/driver/simple/WebSocketClient.java     | 20 ++++---
 pom.xml                                            |  8 +--
 7 files changed, 72 insertions(+), 47 deletions(-)

[tinkerpop] 01/01: Hardcode threadpool size to 2.

Posted by sp...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

spmallette pushed a commit to branch travis-fix
in repository https://gitbox.apache.org/repos/asf/tinkerpop.git

commit e69a17d5c98c0b685206f5a3c7148bb55110ade8
Author: Stephen Mallette <st...@amazon.com>
AuthorDate: Fri Apr 30 06:23:37 2021 -0400

    Hardcode threadpool size to 2.
    
    Travis likely sets the thread pool to 1 for the test client and that may create hanging situations for some reason internal to netty.CTR
---
 .../java/org/apache/tinkerpop/gremlin/driver/simple/AbstractClient.java | 2 +-
 .../org/apache/tinkerpop/gremlin/driver/simple/WebSocketClient.java     | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/gremlin-driver/src/main/java/org/apache/tinkerpop/gremlin/driver/simple/AbstractClient.java b/gremlin-driver/src/main/java/org/apache/tinkerpop/gremlin/driver/simple/AbstractClient.java
index 270277b..6fd5a6d 100644
--- a/gremlin-driver/src/main/java/org/apache/tinkerpop/gremlin/driver/simple/AbstractClient.java
+++ b/gremlin-driver/src/main/java/org/apache/tinkerpop/gremlin/driver/simple/AbstractClient.java
@@ -42,7 +42,7 @@ public abstract class AbstractClient implements SimpleClient {
 
     public AbstractClient(final String threadPattern) {
         final BasicThreadFactory threadFactory = new BasicThreadFactory.Builder().namingPattern(threadPattern).build();
-        group = new NioEventLoopGroup(1, threadFactory);
+        group = new NioEventLoopGroup(2, threadFactory);
     }
 
     public abstract void writeAndFlush(final RequestMessage requestMessage) throws Exception;
diff --git a/gremlin-driver/src/main/java/org/apache/tinkerpop/gremlin/driver/simple/WebSocketClient.java b/gremlin-driver/src/main/java/org/apache/tinkerpop/gremlin/driver/simple/WebSocketClient.java
index 767f5a8..68d1639 100644
--- a/gremlin-driver/src/main/java/org/apache/tinkerpop/gremlin/driver/simple/WebSocketClient.java
+++ b/gremlin-driver/src/main/java/org/apache/tinkerpop/gremlin/driver/simple/WebSocketClient.java
@@ -69,7 +69,7 @@ public class WebSocketClient extends AbstractClient {
 
         try {
             final WebSocketClientHandler wsHandler = new WebSocketClientHandler(WebSocketClientHandshakerFactory.newHandshaker(
-                    uri, WebSocketVersion.V13, null, true, EmptyHttpHeaders.INSTANCE, 65536), 10000);
+                    uri, WebSocketVersion.V13, null, true, EmptyHttpHeaders.INSTANCE, 65536), 15000);
             final MessageSerializer<GraphBinaryMapper> serializer = new GraphBinaryMessageSerializerV1();
             b.channel(NioSocketChannel.class)
                     .handler(new ChannelInitializer<SocketChannel>() {