PULL instead of DISCARD for COMMIT response

Author: technige

driver/src/main/java/org/neo4j/driver/internal/ExplicitTransaction.java

@@ -123,14 +123,14 @@ public void close()
                 if ( state == State.MARKED_SUCCESS )
                 {
                     conn.run( "COMMIT", Collections.<String, Value>emptyMap(), Collector.NO_OP );
-                    conn.discardAll( new BookmarkCollector( this ) );
+                    conn.pullAll( new BookmarkCollector( this ) );
                     conn.sync();
                     state = State.SUCCEEDED;
                 }
                 else if ( state == State.MARKED_FAILED || state == State.ACTIVE )
                 {
                     conn.run( "ROLLBACK", Collections.<String, Value>emptyMap(), Collector.NO_OP );
-                    conn.discardAll( new BookmarkCollector( this ) );
+                    conn.pullAll( new BookmarkCollector( this ) );
                     conn.sync();
                     state = State.ROLLED_BACK;
                 }

driver/src/test/java/org/neo4j/driver/v1/integration/BookmarkIT.java

@@ -28,6 +28,8 @@
 import org.neo4j.driver.v1.util.ServerVersion;
 import org.neo4j.driver.v1.util.TestNeo4jSession;
 
+import static org.hamcrest.MatcherAssert.assertThat;
+import static org.hamcrest.core.StringStartsWith.startsWith;
 import static org.junit.Assert.assertNotNull;
 import static org.junit.Assert.assertNull;
 import static org.junit.Assume.assumeTrue;
@@ -63,5 +65,6 @@ public void shouldReceiveBookmarkOnSuccessfulCommit() throws Throwable
 
         // Then
         assertNotNull( session.lastBookmark() );
+        assertThat( session.lastBookmark(), startsWith( "neo4j:bookmark:v1:tx" ) );
     }
 }