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/06/17 21:25:36 UTC

[tinkerpop] 01/03: TINKERPOP-2238 Fixed iterator leak in SubgraphStrategy test

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

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

commit 32ee56f8cfcbb0f3458789c7da5222e55446b1ae
Author: Stephen Mallette <sp...@genoprime.com>
AuthorDate: Mon Jun 17 16:50:55 2019 -0400

    TINKERPOP-2238 Fixed iterator leak in SubgraphStrategy test
---
 .../traversal/strategy/decoration/SubgraphStrategyProcessTest.java    | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/strategy/decoration/SubgraphStrategyProcessTest.java b/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/strategy/decoration/SubgraphStrategyProcessTest.java
index 0e77966..c7b9f36 100644
--- a/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/strategy/decoration/SubgraphStrategyProcessTest.java
+++ b/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/strategy/decoration/SubgraphStrategyProcessTest.java
@@ -20,7 +20,6 @@ package org.apache.tinkerpop.gremlin.process.traversal.strategy.decoration;
 
 import org.apache.commons.configuration.MapConfiguration;
 import org.apache.tinkerpop.gremlin.LoadGraphWith;
-import org.apache.tinkerpop.gremlin.IgnoreIteratorLeak;
 import org.apache.tinkerpop.gremlin.process.AbstractGremlinProcessTest;
 import org.apache.tinkerpop.gremlin.process.GremlinProcessRunner;
 import org.apache.tinkerpop.gremlin.process.remote.RemoteGraph;
@@ -38,6 +37,7 @@ import org.apache.tinkerpop.gremlin.process.traversal.util.TraversalHelper;
 import org.apache.tinkerpop.gremlin.structure.Column;
 import org.apache.tinkerpop.gremlin.structure.Edge;
 import org.apache.tinkerpop.gremlin.structure.Vertex;
+import org.apache.tinkerpop.gremlin.structure.util.CloseableIterator;
 import org.hamcrest.Matchers;
 import org.junit.Test;
 import org.junit.runner.RunWith;
@@ -62,7 +62,6 @@ import static org.junit.Assume.assumeThat;
  * @author Stephen Mallette (http://stephen.genoprime.com)
  */
 @RunWith(GremlinProcessRunner.class)
-@IgnoreIteratorLeak
 public class SubgraphStrategyProcessTest extends AbstractGremlinProcessTest {
 
     @Test
@@ -164,6 +163,7 @@ public class SubgraphStrategyProcessTest extends AbstractGremlinProcessTest {
         final Traversal t = sg.V();
         t.hasNext();
         printTraversalForm(t);
+        CloseableIterator.closeIterator(t);
         assertEquals(6, sg.V().count().next().longValue());
 
         // only the given edges are included