[tinkerpop] branch docker-test-server updated (c8cb5eb -> e498ab8)
This is an automated email from the ASF dual-hosted git repository. rdale pushed a change to branch docker-test-server in repository https://gitbox.apache.org/repos/asf/tinkerpop.git. discard c8cb5eb added docker gremlin-server test script add e498ab8 added docker gremlin-server test script 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 (c8cb5eb) \ N -- N -- N refs/heads/docker-test-server (e498ab8) 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. No new revisions were added by this update. Summary of changes: docker/gremlin-server/run.sh | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-)
[GitHub] spmallette opened a new pull request #1076: TINKERPOP-2112 Fold property() so that T values can work in any order
spmallette opened a new pull request #1076: TINKERPOP-2112 Fold property() so that T values can work in any order URL: https://github.com/apache/tinkerpop/pull/1076 Also optimizes a bit to capture additional possible folds to the "AddElementStep" that would have been otherwise missed before. This should enhance performance of certain mutation traversals depending on the positioning and parameterization of the property() steps as more folding will occur. This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] spmallette commented on issue #1057: Changed loop break check to break out of the connection creation loop
spmallette commented on issue #1057: Changed loop break check to break out of the connection creation loop URL: https://github.com/apache/tinkerpop/pull/1057#issuecomment-468712877 We have been floating the idea of a release for mid-march - so a couple of weeks or so. This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[tinkerpop] branch TINKERPOP-2112 updated (afa65af -> 1fd6a91)
This is an automated email from the ASF dual-hosted git repository. spmallette pushed a change to branch TINKERPOP-2112 in repository https://gitbox.apache.org/repos/asf/tinkerpop.git. discard afa65af TINKERPOP-2112 Fixed folding in property() step add f9d7027 Added Cluster.using(Client) CTR add 79b5e1c TINKERPOP-2096 Throw GremlinServerError if server disconnects before read completes add fa5e5b6 spelling CTR add 689e034 TINKERPOP-2110 Added option to change path for java driver add b711c7e Merge branch 'TINKERPOP-2110' into tp33 add c0a93ec Fixed aliasing for the new using() overload CTR add fb734ea Fixed bug preventing TraversalExplanation from serializing with GraphSON CTR add d58712c TINKERPOP-1889 - Upgrade ws, adding ping and reconnection javascript add ccd6236 TINKERPOP-2113 Fix P.Within() for list arguments add 6b51c54 fix for TINKERPOP-2105 add c0d62b8 Merge branch 'pr-1023' into tp33 add d548d42 Merged template change without regenerating package.json add b360463 Bump Jackson Databind 2.9.8 add 48014ce Merge branch 'tp32' into tp33 add e9dd3cc TINKERPOP-2118 Bump to groovy 2.4.16 add 58210af Update changelog for 3.2.11 release CTR add 6026ac0 Update for 3.2.11 upgrade docs CTR add ed94a75 TinkerPop 3.2.11 release add 7110d26 Merge branch 'tp32' into tp33 add 277194c TinkerPop 3.2.11 release add 437b0f6 Fixed 2018 nonsense for release date CTR add b75a703 fixed link - CTR add 28b6322 Override the apache-release profile add dfc88b4 CTR: create .glv files before building project in validation script add a21001a Add .glv to gremlin-javascript for docker builds CTR add 82f0590 Remove latest tag from tp33 TINKERPOP-1980 CTR add 60fd135 Bump to 3.3.6-SNAPSHOT add 71a58ce Added link to .NET docs on compendium page CTR add 12381ae Minor grammatical fix in error message CTR add dc92a3c Added js/.net api docs to the vote thread email template CTR add 3f02660 Added Gremlin.Net.Template to the nuget links to check during release CTR add 6588e10 Fix minor grammatical error in release docs CTR add c806510 Replaced deprecated tag in antrun plugins CTR add c0779e2 Massively cut down log output with new radish feature CTR add fa0afb5 Fixed warnings related to nuget around the license CTR add 02a7af3 Mask security secret or password add 42bdff0 TINKERPOP-2126 Made `TraverserSet` internals thread-safe add 93cd399 Merge branch 'TINKERPOP-2126' into tp33 add 48f1332 TINKERPOP-2124 Fixed and/or folding in `InlineFilterStrategy`. add da2e32f Merge pull request #1030 from apache/TINKERPOP-2124 add 346d984 TINKERPOP-2130 Fix default parameter assignment in Connection ctor add e23cf5e Merge pull request #1038 from nivsherf/TINKERPOP-2130 add 2bcf8b5 Removed is.testing config option from remote traversals add b57ac11 TINKERPOP-2088 Enable SourceLink for Gremlin.Net add f066371 Merge branch 'TINKERPOP-2088' into tp33 add e3922b8 TINKERPOP-2136 Fixed explanation of the `inside` predicate bounds. add 901b93c Added test data files for 3.3.6 CTR add 35e870e Fix typo in bash script: tue -> true CTR add b7cf0d4 Add Python TraversalMetrics/Metrics deserializers add b565908 Remove print statements in tests add 86517fa Merge branch 'pr-1031' into tp33 add 02ad4be TINKERPOP-2140 Add CI smoke test for Docker build add 5f7bc22 Merge branch 'TINKERPOP-2140' into tp33 add 197188e TINKERPOP-2138 Provide way to disable the global functionc cache add fe60010 Merge branch 'TINKERPOP-2138' into tp33 add c9dbcf0 Fixed bad import declaration in python docs CTR add ebd7f26 Implement gherkin tests that required GraphSON 3.0 add 92d313d TINKERPOP-1998 Minor fix to javadoc around the test suite and added some notes to the provider dev docs add ff95667 TINKERPOP-2125 Added source file check in release validation script add 4d583b4 Merge pull request #1041 from apache/TINKERPOP-2125 add 2512d691 Fixed bad test assertions in gherkin add 38dcfb4 Ignored some failing .NET tests after GLV tests were enabled for GraphSON 3.0 CTR add 40e3e2b TINKERPOP-2122 Expose the status code on GremlinServerError in python CTR add a004b72 TINKERPOP-2139 Properly handle client side serialization exceptions add 62622c2 TINKERPOP-2126 Fixed concurrency issue in `ObjectWritable::toString()`. add fec6573 Merge pull request #1044 from apache/TINKERPOP-2126 add 996b1ee Release dropped response Frames in ResponseHandlerContext add 89acb65 Merge branch 'pr-1052' into tp33 add d523bb4 Bumped jacoco/surefire plugin and set minimum maven version CTR add 8ca76cb Bump the revapi plugin CTR add 47f6ae0 Bumped the .NET maven plugin to 0.23
[tinkerpop] 01/01: TINKERPOP-2112 Fold property() so that T values can work in any order
This is an automated email from the ASF dual-hosted git repository.
spmallette pushed a commit to branch TINKERPOP-2112
in repository https://gitbox.apache.org/repos/asf/tinkerpop.git
commit 1fd6a91defb1781b7a916894bb9ef533f77c8f9d
Author: Stephen Mallette
AuthorDate: Fri Mar 1 10:42:05 2019 -0500
TINKERPOP-2112 Fold property() so that T values can work in any order
Also optimizes a bit to capture additional possible folds to the
"AddElementStep" that would have been otherwise missed before. This should
enhance performance of certain mutation traversals depending on the positioning
and parameterization of the property() steps as more folding will occur.
---
CHANGELOG.asciidoc| 1 +
.../process/traversal/dsl/graph/GraphTraversal.java | 16 ++--
.../tinkergraph/structure/TinkerGraphTest.java| 19 +++
3 files changed, 34 insertions(+), 2 deletions(-)
diff --git a/CHANGELOG.asciidoc b/CHANGELOG.asciidoc
index a369365..0b1f3c5 100644
--- a/CHANGELOG.asciidoc
+++ b/CHANGELOG.asciidoc
@@ -38,6 +38,7 @@
image::https://raw.githubusercontent.com/apache/tinkerpop/master/docs/static/ima
* Bumped `httpclient` to 4.5.7.
* Bumped `slf4j` to 1.7.25.
* Bumped `commons-codec` to 1.12.
+* Improved folding of `property()` step into related mutating steps.
* Fixed concurrency issues in `TraverserSet.toString()` and
`ObjectWritable.toString()`.
* Fixed a bug in `InlineFilterStrategy` that mixed up and's and or's when
folding merging conditions together.
* Improved handling of failing `Authenticator` instances thus improving server
responses to drivers.
diff --git
a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/dsl/graph/GraphTraversal.java
b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/dsl/graph/GraphTraversal.java
index 92593c3..62ed152 100644
---
a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/dsl/graph/GraphTraversal.java
+++
b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/dsl/graph/GraphTraversal.java
@@ -2081,8 +2081,20 @@ public interface GraphTraversal extends
Traversal {
// if it can be detected that this call to property() is related to an
addV/E() then we can attempt to fold
// the properties into that step to gain an optimization for those
graphs that support such capabilities.
-final Step endStep = this.asAdmin().getEndStep();
-if ((endStep instanceof AddVertexStep || endStep instanceof
AddEdgeStep || endStep instanceof AddVertexStartStep || endStep instanceof
AddEdgeStartStep) &&
+Step endStep = this.asAdmin().getEndStep();
+
+// always try to fold the property() into the initial "AddElementStep"
as the performance will be better
+// and as it so happens with T the value must be set by way of that
approach otherwise you get an error.
+// it should be safe to execute this loop this way as we'll either hit
an "AddElementStep" or an "EmptyStep".
+// if empty, it will just use the regular AddPropertyStep being tacked
on to the end of the traversal as usual
+while (endStep instanceof AddPropertyStep) {
+endStep = endStep.getPreviousStep();
+}
+
+// it's possible to fold the property() into the Mutating step if
there are no metaproperties (i.e. keyValues)
+// and if the cardinality is unspecified
+if ((endStep instanceof AddVertexStep || endStep instanceof
AddEdgeStep ||
+endStep instanceof AddVertexStartStep || endStep instanceof
AddEdgeStartStep) &&
keyValues.length == 0 && null == cardinality) {
((Mutating) endStep).addPropertyMutations(key, value);
} else {
diff --git
a/tinkergraph-gremlin/src/test/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerGraphTest.java
b/tinkergraph-gremlin/src/test/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerGraphTest.java
index a5efcb8..a4408af 100644
---
a/tinkergraph-gremlin/src/test/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerGraphTest.java
+++
b/tinkergraph-gremlin/src/test/java/org/apache/tinkerpop/gremlin/tinkergraph/structure/TinkerGraphTest.java
@@ -75,6 +75,7 @@ import java.util.function.Supplier;
import static org.hamcrest.MatcherAssert.assertThat;
import static org.hamcrest.Matchers.greaterThan;
+import static org.hamcrest.Matchers.is;
import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertNotSame;
import static org.junit.Assert.assertTrue;
@@ -650,6 +651,24 @@ public class TinkerGraphTest {
}
/**
+ * Just validating that property folding works nicely given TINKERPOP-2112
+ */
+@Test
+public void shouldFoldPropertyStepForTokens() {
+final GraphTraversalSource g = TinkerGraph.open().traversal();
+
+
[GitHub] tomek22 commented on issue #1057: Changed loop break check to break out of the connection creation loop
tomek22 commented on issue #1057: Changed loop break check to break out of the connection creation loop URL: https://github.com/apache/tinkerpop/pull/1057#issuecomment-468709823 I just spent almost a full day figuring out why our service leaks memory and can't open sockets. I tracked it down to this issue. Managed to reproduce locally by killing the gremlin server after the connection pool has already been created and then restarting gremlin server. I was very happy to see it fixed in master, but sad to see it hasn't been released yet. Any idea when there will be a nuget package available with this fix? We had to workaround the issue by moving the gremlin client to request scope instead of singleton which I expect will result in some performance penalty :( This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[tinkerpop] branch docker-test-server updated (be1bb36 -> c8cb5eb)
This is an automated email from the ASF dual-hosted git repository. rdale pushed a change to branch docker-test-server in repository https://gitbox.apache.org/repos/asf/tinkerpop.git. discard be1bb36 added docker gremlin-server test script add c8cb5eb added docker gremlin-server test script 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 (be1bb36) \ N -- N -- N refs/heads/docker-test-server (c8cb5eb) 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. No new revisions were added by this update. Summary of changes: docker/gremlin-server/run.sh | 19 +++ 1 file changed, 19 insertions(+)
[GitHub] robertdale opened a new pull request #1075: Docker gremlin-server test script
robertdale opened a new pull request #1075: Docker gremlin-server test script URL: https://github.com/apache/tinkerpop/pull/1075 [GremlinDev discussion](https://lists.apache.org/thread.html/b3cbdd9083a981681bf6bf0a540dc9ae35d78617dd8d278a53f0995e@%3Cdev.tinkerpop.apache.org%3E) This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[tinkerpop] 01/01: added docker gremlin-server test script
This is an automated email from the ASF dual-hosted git repository.
rdale pushed a commit to branch docker-test-server
in repository https://gitbox.apache.org/repos/asf/tinkerpop.git
commit be1bb364df3c5871cc10c68b4ccf979edcea231b
Author: Robert Dale
AuthorDate: Fri Mar 1 09:57:39 2019 -0500
added docker gremlin-server test script
---
docker/gremlin-server.sh | 55 +++
docker/gremlin-server/Dockerfile.template | 27 +++
docker/gremlin-server/run.sh | 8 +
3 files changed, 90 insertions(+)
diff --git a/docker/gremlin-server.sh b/docker/gremlin-server.sh
new file mode 100755
index 000..0a414e7
--- /dev/null
+++ b/docker/gremlin-server.sh
@@ -0,0 +1,55 @@
+#!/bin/bash
+#
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements. See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership. The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance
+# with the License. You may obtain a copy of the License at
+#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+# KIND, either express or implied. See the License for the
+# specific language governing permissions and limitations
+# under the License.
+#
+
+# Build docker images first
+# Build gremlin-server: mvn clean install -pl :gremlin-server -am
+
+DIR=`dirname $0`
+PROJECT_HOME=${DIR}/../
+
+TIMESTAMP=$(date +%s)
+BUILD_TAG="gremlin-server-test-${TIMESTAMP}"
+
+function cleanup {
+ BUILD_IMAGE=$(docker images tinkerpop | awk "{if (\$2 == \"${BUILD_TAG}\")
print \$3}")
+ [ ! -z ${BUILD_IMAGE} ] && docker rmi ${BUILD_IMAGE}
+ rm -f ${PROJECT_HOME}/Dockerfile
+}
+trap cleanup EXIT
+
+REMOVE_CONTAINER="--rm"
+[[ -n ${KEEP_CONTAINER} ]] && unset REMOVE_CONTAINER
+
+pushd ${PROJECT_HOME} > /dev/null
+
+#mvn clean package -DskipTests=true -pl :gremlin-server || mvn clean install
-pl :gremlin-server -am
+
+HADOOP_VERSION=$(cat pom.xml | grep -o '[^<]*' | head -n1 |
cut -d '>' -f2)
+
+docker/build-containers.sh -h "${HADOOP_VERSION}"
+
+sed -e "s/HADOOP_VERSION\$/${HADOOP_VERSION}/"
docker/gremlin-server/Dockerfile.template > Dockerfile
+
+docker build -t tinkerpop:${BUILD_TAG} .
+docker run ${TINKERPOP_DOCKER_OPTS} ${REMOVE_CONTAINER} -ti
tinkerpop:${BUILD_TAG}
+
+status=$?
+popd > /dev/null
+exit $status
diff --git a/docker/gremlin-server/Dockerfile.template
b/docker/gremlin-server/Dockerfile.template
new file mode 100644
index 000..ef3d17e
--- /dev/null
+++ b/docker/gremlin-server/Dockerfile.template
@@ -0,0 +1,27 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements. See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership. The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance
+# with the License. You may obtain a copy of the License at
+#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+# KIND, either express or implied. See the License for the
+# specific language governing permissions and limitations
+# under the License.
+
+FROM tinkerpop:hadoop-HADOOP_VERSION
+
+RUN mkdir -p /opt
+WORKDIR /opt
+COPY gremlin-server/target/apache-tinkerpop-gremlin-server-*-distribution.zip
/opt/
+RUN unzip *.zip && rm *.zip
+COPY gremlin-server/src/test /opt/test/
+COPY docker/gremlin-server/run.sh /opt/
+RUN chmod 755 /opt/run.sh
+CMD /opt/run.sh
\ No newline at end of file
diff --git a/docker/gremlin-server/run.sh b/docker/gremlin-server/run.sh
new file mode 100644
index 000..4d661f0
--- /dev/null
+++ b/docker/gremlin-server/run.sh
@@ -0,0 +1,8 @@
+
+TINKERPOP_HOME=$(find . -type d -name 'apache-tinkerpop-gremlin-server-*'
-print -quit)
+
+cp /opt/test/scripts/* ${TINKERPOP_HOME}/scripts
+cp
/opt/test/resources/org/apache/tinkerpop/gremlin/server/gremlin-server-integration.yaml
${TINKERPOP_HOME}/conf
+
+cd ${TINKERPOP_HOME}
+./bin/gremlin-server.sh conf/gremlin-server-integration.yaml
[tinkerpop] branch docker-test-server created (now be1bb36)
This is an automated email from the ASF dual-hosted git repository. rdale pushed a change to branch docker-test-server in repository https://gitbox.apache.org/repos/asf/tinkerpop.git. at be1bb36 added docker gremlin-server test script This branch includes the following new commits: new be1bb36 added docker gremlin-server test script 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.
[GitHub] joestrouth1 commented on issue #1071: TINKERPOP-2167 Traversals as async iterables in gremlin-javascript
joestrouth1 commented on issue #1071: TINKERPOP-2167 Traversals as async iterables in gremlin-javascript URL: https://github.com/apache/tinkerpop/pull/1071#issuecomment-468676653 @jorgebay Sure thing, I should be able to get to it this weekend. This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[GitHub] harsh9t commented on issue #1074: TINKERPOP-2171 Allow sparql() to be extended with Gremlin steps.
harsh9t commented on issue #1074: TINKERPOP-2171 Allow sparql() to be extended with Gremlin steps. URL: https://github.com/apache/tinkerpop/pull/1074#issuecomment-468646203 VOTE +1 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[tinkerpop] branch TINKERPOP-2171 updated (c368068 -> 984f086)
This is an automated email from the ASF dual-hosted git repository. spmallette pushed a change to branch TINKERPOP-2171 in repository https://gitbox.apache.org/repos/asf/tinkerpop.git. discard c368068 TINKERPOP-2171 Allow sparql() to be extended with Gremlin steps. add f139b40 TINKERPOP-1992 Fix negative time on profile() step add f980886 TINKERPOP-1992 Update changelog add ee30283 TINKERPOP-1992 Added test to validate that durations are greater than zero. add 44693d0 TINKERPOP-1992 Prevented reset of barrier from happening more than once add 29fe4cd Merge branch 'TINKERPOP-1992' into tp33 add 72eb72f Merge branch 'tp33' add 5ac3add Added "note" to .net project template. add 3c87b3a Merge branch 'tp33' add 543a0e2 TINKERPOP-2166 Cache expression to obtain the method in PSerializer add e02b8fc TINKERPOP-2166 Changelog add 2e84dfb Merge pull request #1073 from apache/TINKERPOP-2166 add 984f086 TINKERPOP-2171 Allow sparql() to be extended with Gremlin steps. 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 (c368068) \ N -- N -- N refs/heads/TINKERPOP-2171 (984f086) 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. No new revisions were added by this update. Summary of changes: CHANGELOG.asciidoc | 3 + .../process/traversal/step/ProfilingAware.java | 98 +++ .../process/traversal/step/map/GroupStep.java | 39 +- .../step/sideEffect/GroupSideEffectStep.java | 24 +++- .../process/traversal/step/util/ProfileStep.java | 15 +++ .../strategy/finalization/ProfileStrategy.java | 9 +- gremlin-dotnet/glv/Gremlin.Net.csproj.template | 4 +- .../driver/ser/binary/types/PSerializer.java | 133 +++-- .../driver/GraphBinaryReaderWriterBenchmark.java | 9 ++ .../tinkergraph/structure/TinkerGraphTest.java | 25 10 files changed, 318 insertions(+), 41 deletions(-) create mode 100644 gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/ProfilingAware.java
[tinkerpop] branch master updated (3c87b3a -> 2e84dfb)
This is an automated email from the ASF dual-hosted git repository. jorgebg pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/tinkerpop.git. from 3c87b3a Merge branch 'tp33' new 543a0e2 TINKERPOP-2166 Cache expression to obtain the method in PSerializer new e02b8fc TINKERPOP-2166 Changelog new 2e84dfb Merge pull request #1073 from apache/TINKERPOP-2166 The 14997 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 +- .../driver/ser/binary/types/PSerializer.java | 133 +++-- .../driver/GraphBinaryReaderWriterBenchmark.java | 9 ++ 3 files changed, 109 insertions(+), 35 deletions(-)
[tinkerpop] branch TINKERPOP-2166 deleted (was e02b8fc)
This is an automated email from the ASF dual-hosted git repository. jorgebg pushed a change to branch TINKERPOP-2166 in repository https://gitbox.apache.org/repos/asf/tinkerpop.git. was e02b8fc TINKERPOP-2166 Changelog The revisions that were on this branch are still contained in other references; therefore, this change does not discard any commits from the repository.
[GitHub] jorgebay merged pull request #1073: TINKERPOP-2166 Cache expression to obtain the method in PSerializer
jorgebay merged pull request #1073: TINKERPOP-2166 Cache expression to obtain the method in PSerializer URL: https://github.com/apache/tinkerpop/pull/1073 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[tinkerpop] branch TINKERPOP-2166 updated: TINKERPOP-2166 Changelog
This is an automated email from the ASF dual-hosted git repository. jorgebg pushed a commit to branch TINKERPOP-2166 in repository https://gitbox.apache.org/repos/asf/tinkerpop.git The following commit(s) were added to refs/heads/TINKERPOP-2166 by this push: new e02b8fc TINKERPOP-2166 Changelog e02b8fc is described below commit e02b8fc2cac596373645666f1b59881791352698 Author: Jorge Bay Gondra AuthorDate: Fri Mar 1 10:09:06 2019 +0100 TINKERPOP-2166 Changelog --- CHANGELOG.asciidoc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/CHANGELOG.asciidoc b/CHANGELOG.asciidoc index ea8205f..d22cfbf 100644 --- a/CHANGELOG.asciidoc +++ b/CHANGELOG.asciidoc @@ -37,7 +37,7 @@ This release also includes changes from <>. * Release working buffers in case of failure for GraphBinary. * GraphBinary: Use the same `ByteBuf` instance to write during serialization. Changed signature of write methods in type serializers. * Remove unused parameter in GraphBinary's `ResponseMessageSerializer`. - +* GraphBinary: Cache expression to obtain the method in `PSerializer`. [[release-3-4-0]] === TinkerPop 3.4.0 (Release Date: January 2, 2019)
[GitHub] jorgebay commented on issue #1071: TINKERPOP-2167 Traversals as async iterables in gremlin-javascript
jorgebay commented on issue #1071: TINKERPOP-2167 Traversals as async iterables in gremlin-javascript URL: https://github.com/apache/tinkerpop/pull/1071#issuecomment-468594110 @spmallette You are right, it does make sense to deliver it on `3.3.x`... @joestrouth1 Do you mind basing your pull request on `tp33` so this can land on both `3.3` and `3.4`? In case you don't have time to do it, I could cherry pick it. This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
