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/02 12:34:23 UTC
[tinkerpop] 06/06: TINKERPOP-2245 gremlinPool needs to be defaulted
to a larger value
This is an automated email from the ASF dual-hosted git repository.
spmallette pushed a commit to branch TINKERPOP-2245
in repository https://gitbox.apache.org/repos/asf/tinkerpop.git
commit dd55326654b67631691ae313ad9cc0fc57a5741f
Author: Stephen Mallette <st...@amazon.com>
AuthorDate: Fri Apr 2 06:07:34 2021 -0400
TINKERPOP-2245 gremlinPool needs to be defaulted to a larger value
For environments like travis which might only have two cores there is a chance for tests to fail/not finish as they introduce blocks in requests that need to exist for tests to pass. If there aren't enough threads in the pool the requests can't be serviced properly and they fail.
---
.travis.yml | 2 +-
docker/gremlin-server/gremlin-server-integration-krb5.yaml | 1 +
docker/gremlin-server/gremlin-server-integration-secure.yaml | 1 +
docker/gremlin-server/gremlin-server-integration.yaml | 1 +
.../org/apache/tinkerpop/gremlin/server/gremlin-server-integration.yaml | 2 ++
5 files changed, 6 insertions(+), 1 deletion(-)
diff --git a/.travis.yml b/.travis.yml
index ff3a058..b8b02a7 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -60,7 +60,7 @@ jobs:
name: "gremlin server"
- script:
- "mvn clean install -q -DskipTests -Dci"
- - "travis_wait 60 mvn verify -pl :gremlin-server -DskipTests -DskipIntegrationTests=false -DincludeNeo4j -DtestUnified=true"
+ - "mvn verify -pl :gremlin-server -DskipTests -DskipIntegrationTests=false -DincludeNeo4j -DtestUnified=true"
name: "gremlin server - unified"
- script:
- "mvn clean install -q -DskipTests -Dci"
diff --git a/docker/gremlin-server/gremlin-server-integration-krb5.yaml b/docker/gremlin-server/gremlin-server-integration-krb5.yaml
index 16bad46..cb7e737 100644
--- a/docker/gremlin-server/gremlin-server-integration-krb5.yaml
+++ b/docker/gremlin-server/gremlin-server-integration-krb5.yaml
@@ -48,6 +48,7 @@ processors:
- { className: org.apache.tinkerpop.gremlin.server.op.standard.StandardOpProcessor, config: {}}
metrics: {
slf4jReporter: {enabled: true, interval: 180000}}
+gremlinPool: 8
strictTransactionManagement: false
idleConnectionTimeout: 0
keepAliveInterval: 0
diff --git a/docker/gremlin-server/gremlin-server-integration-secure.yaml b/docker/gremlin-server/gremlin-server-integration-secure.yaml
index 1cd3156..e6a999f 100644
--- a/docker/gremlin-server/gremlin-server-integration-secure.yaml
+++ b/docker/gremlin-server/gremlin-server-integration-secure.yaml
@@ -48,6 +48,7 @@ processors:
- { className: org.apache.tinkerpop.gremlin.server.op.standard.StandardOpProcessor, config: {}}
metrics: {
slf4jReporter: {enabled: true, interval: 180000}}
+gremlinPool: 8
strictTransactionManagement: false
idleConnectionTimeout: 0
keepAliveInterval: 0
diff --git a/docker/gremlin-server/gremlin-server-integration.yaml b/docker/gremlin-server/gremlin-server-integration.yaml
index 13c2d6c..7f56673 100644
--- a/docker/gremlin-server/gremlin-server-integration.yaml
+++ b/docker/gremlin-server/gremlin-server-integration.yaml
@@ -48,6 +48,7 @@ processors:
- { className: org.apache.tinkerpop.gremlin.server.op.standard.StandardOpProcessor, config: {}}
metrics: {
slf4jReporter: {enabled: true, interval: 180000}}
+gremlinPool: 8
strictTransactionManagement: false
idleConnectionTimeout: 0
keepAliveInterval: 0
diff --git a/gremlin-server/src/test/resources/org/apache/tinkerpop/gremlin/server/gremlin-server-integration.yaml b/gremlin-server/src/test/resources/org/apache/tinkerpop/gremlin/server/gremlin-server-integration.yaml
index 601e404..35a19d7 100644
--- a/gremlin-server/src/test/resources/org/apache/tinkerpop/gremlin/server/gremlin-server-integration.yaml
+++ b/gremlin-server/src/test/resources/org/apache/tinkerpop/gremlin/server/gremlin-server-integration.yaml
@@ -21,6 +21,7 @@
# Changes to this file need to be appropriately replicated to
#
# - docker/gremlin-server/gremlin-server-integration.yaml
+# - docker/gremlin-server/gremlin-server-integration-krb5.yaml
# - docker/gremlin-server/gremlin-server-integration-secure.yaml
#
# Without such changes, the test docker server can't be started for independent
@@ -61,6 +62,7 @@ processors:
- { className: org.apache.tinkerpop.gremlin.server.op.standard.StandardOpProcessor, config: {}}
metrics: {
slf4jReporter: {enabled: true, interval: 180000}}
+gremlinPool: 8
strictTransactionManagement: false
idleConnectionTimeout: 0
keepAliveInterval: 0