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 2019/04/16 17:59:57 UTC

[tinkerpop] branch master updated: Fix threaded transaction tests. The graph thread should be refreshed to observe the result of committed transaction.

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 2446825  Fix threaded transaction tests. The graph thread should be refreshed to observe the result of committed transaction.
     new d32d42e  Merge branch 'tp33'
2446825 is described below

commit 2446825ad0f1e800807cf249fc9baa03209d149d
Author: Hieu Nguyen <hi...@ebay.com>
AuthorDate: Thu Apr 11 23:58:48 2019 -0700

    Fix threaded transaction tests. The graph thread should be refreshed to observe the result of committed transaction.
---
 .../java/org/apache/tinkerpop/gremlin/structure/TransactionTest.java    | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/structure/TransactionTest.java b/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/structure/TransactionTest.java
index ad08132..d7a651b 100644
--- a/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/structure/TransactionTest.java
+++ b/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/structure/TransactionTest.java
@@ -598,6 +598,7 @@ public class TransactionTest extends AbstractGremlinTest {
         threadedG.tx().commit();
 
         // there should be one vertex for each thread
+        graph.tx().rollback();
         assertVertexEdgeCounts(graph, numberOfThreads, 0);
     }
 
@@ -638,6 +639,7 @@ public class TransactionTest extends AbstractGremlinTest {
         threadedG.tx().commit();
 
         // there should be one vertex for each thread
+        graph.tx().rollback();
         assertVertexEdgeCounts(graph, numberOfThreads, 0);
 
         try {