Skip to content

Commit 713c221

Browse files
committedFeb 18, 2025
rename close to finalizeTransaction
1 parent 77b1177 commit 713c221

File tree

8 files changed

+19
-19
lines changed

8 files changed

+19
-19
lines changed
 

‎src/main/java/org/neo4j/gis/spatial/EditableLayer.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -63,5 +63,5 @@ public interface EditableLayer extends Layer {
6363
/**
6464
* Do any cleanup or final calculation required by the layer implementation.
6565
*/
66-
void close(Transaction tx);
66+
void finalizeTransaction(Transaction tx);
6767
}

‎src/main/java/org/neo4j/gis/spatial/EditableLayerImpl.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -83,7 +83,7 @@ public String getSignature() {
8383
}
8484

8585
@Override
86-
public void close(Transaction tx) {
87-
getIndex().close(tx);
86+
public void finalizeTransaction(Transaction tx) {
87+
getIndex().finalizeTransaction(tx);
8888
}
8989
}

‎src/main/java/org/neo4j/gis/spatial/ShapefileImporter.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -221,7 +221,7 @@ record = shpReader.nextRecord();
221221
}
222222
}
223223
try (Transaction tx = database.beginTx()) {
224-
layer.close(tx);
224+
layer.finalizeTransaction(tx);
225225
tx.commit();
226226
}
227227

‎src/main/java/org/neo4j/gis/spatial/index/SpatialIndexReader.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,6 +48,6 @@ public interface SpatialIndexReader {
4848

4949
void configure(Map<String, Object> config);
5050

51-
default void close(Transaction tx) {
51+
default void finalizeTransaction(Transaction tx) {
5252
}
5353
}

‎src/main/java/org/neo4j/gis/spatial/osm/OSMImporter.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -329,7 +329,7 @@ public long reIndex(GraphDatabaseService database, int commitInterval, boolean i
329329
}
330330
} // TODO ask charset to user?
331331
}
332-
layer.close(tx);
332+
layer.finalizeTransaction(tx);
333333
tx.commit();
334334
} finally {
335335
endProgressMonitor();

‎src/main/java/org/neo4j/gis/spatial/procedures/SpatialProcedures.java

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -555,7 +555,7 @@ public void removeLayer(@Name("name") String name) {
555555
public Stream<NodeResult> addNodeToLayer(@Name("layerName") String name, @Name("node") Node node) {
556556
EditableLayer layer = getEditableLayerOrThrow(tx, spatial(), name);
557557
Node geomNode = layer.add(tx, node).getGeomNode();
558-
layer.close(tx);
558+
layer.finalizeTransaction(tx);
559559
return streamNode(geomNode);
560560
}
561561

@@ -564,7 +564,7 @@ public Stream<NodeResult> addNodeToLayer(@Name("layerName") String name, @Name("
564564
public Stream<CountResult> addNodesToLayer(@Name("layerName") String name, @Name("nodes") List<Node> nodes) {
565565
EditableLayer layer = getEditableLayerOrThrow(tx, spatial(), name);
566566
int count = layer.addAll(tx, nodes);
567-
layer.close(tx);
567+
layer.finalizeTransaction(tx);
568568
return Stream.of(new CountResult(count));
569569
}
570570

@@ -573,7 +573,7 @@ public Stream<CountResult> addNodesToLayer(@Name("layerName") String name, @Name
573573
public Stream<NodeResult> addNodeIdToLayer(@Name("layerName") String name, @Name("nodeId") String nodeId) {
574574
EditableLayer layer = getEditableLayerOrThrow(tx, spatial(), name);
575575
Node geomNode = layer.add(tx, tx.getNodeByElementId(nodeId)).getGeomNode();
576-
layer.close(tx);
576+
layer.finalizeTransaction(tx);
577577
return streamNode(geomNode);
578578
}
579579

@@ -584,7 +584,7 @@ public Stream<CountResult> addNodeIdsToLayer(@Name("layerName") String name,
584584
EditableLayer layer = getEditableLayerOrThrow(tx, spatial(), name);
585585
List<Node> nodes = nodeIds.stream().map(id -> tx.getNodeByElementId(id)).collect(Collectors.toList());
586586
int count = layer.addAll(tx, nodes);
587-
layer.close(tx);
587+
layer.finalizeTransaction(tx);
588588
return Stream.of(new CountResult(count));
589589
}
590590

@@ -593,7 +593,7 @@ public Stream<CountResult> addNodeIdsToLayer(@Name("layerName") String name,
593593
public Stream<NodeIdResult> removeNodeFromLayer(@Name("layerName") String name, @Name("node") Node node) {
594594
EditableLayer layer = getEditableLayerOrThrow(tx, spatial(), name);
595595
layer.removeFromIndex(tx, node.getElementId());
596-
layer.close(tx);
596+
layer.finalizeTransaction(tx);
597597
return streamNode(node.getElementId());
598598
}
599599

@@ -607,7 +607,7 @@ public Stream<CountResult> removeNodesFromLayer(@Name("layerName") String name,
607607
layer.removeFromIndex(tx, node.getElementId());
608608
}
609609
int after = layer.getIndex().count(tx);
610-
layer.close(tx);
610+
layer.finalizeTransaction(tx);
611611
return Stream.of(new CountResult(before - after));
612612
}
613613

@@ -616,7 +616,7 @@ public Stream<CountResult> removeNodesFromLayer(@Name("layerName") String name,
616616
public Stream<NodeIdResult> removeNodeFromLayer(@Name("layerName") String name, @Name("nodeId") String nodeId) {
617617
EditableLayer layer = getEditableLayerOrThrow(tx, spatial(), name);
618618
layer.removeFromIndex(tx, nodeId);
619-
layer.close(tx);
619+
layer.finalizeTransaction(tx);
620620
return streamNode(nodeId);
621621
}
622622

@@ -631,7 +631,7 @@ public Stream<CountResult> removeNodeIdsFromLayer(@Name("layerName") String name
631631
layer.removeFromIndex(tx, nodeId);
632632
}
633633
int after = layer.getIndex().count(tx);
634-
layer.close(tx);
634+
layer.finalizeTransaction(tx);
635635
return Stream.of(new CountResult(before - after));
636636
}
637637

@@ -642,7 +642,7 @@ public Stream<NodeResult> addGeometryWKTToLayer(@Name("layerName") String name,
642642
EditableLayer layer = getEditableLayerOrThrow(tx, spatial(), name);
643643
WKTReader reader = new WKTReader(layer.getGeometryFactory());
644644
Node node = addGeometryWkt(layer, reader, geometryWKT);
645-
layer.close(tx);
645+
layer.finalizeTransaction(tx);
646646
return streamNode(node);
647647
}
648648

@@ -654,7 +654,7 @@ public Stream<NodeResult> addGeometryWKTsToLayer(@Name("layerName") String name,
654654
WKTReader reader = new WKTReader(layer.getGeometryFactory());
655655
return geometryWKTs.stream().map(geometryWKT -> addGeometryWkt(layer, reader, geometryWKT))
656656
.map(NodeResult::new)
657-
.onClose(() -> layer.close(tx));
657+
.onClose(() -> layer.finalizeTransaction(tx));
658658
}
659659

660660
private Node addGeometryWkt(EditableLayer layer, WKTReader reader, String geometryWKT) {
@@ -673,7 +673,7 @@ public Stream<CountResult> importShapefile(
673673
@Name("uri") String uri) throws IOException {
674674
EditableLayerImpl layer = getEditableLayerOrThrow(tx, spatial(), name);
675675
List<Node> nodes = importShapefileToLayer(uri, layer, 1000);
676-
layer.close(tx);
676+
layer.finalizeTransaction(tx);
677677
return Stream.of(new CountResult(nodes.size()));
678678
}
679679

‎src/main/java/org/neo4j/gis/spatial/rtree/RTreeIndex.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1556,7 +1556,7 @@ public int compare(NodeWithEnvelope o1, NodeWithEnvelope o2) {
15561556
}
15571557

15581558
@Override
1559-
public void close(Transaction tx) {
1559+
public void finalizeTransaction(Transaction tx) {
15601560
saveCount(tx);
15611561
}
15621562
}

‎src/test/java/org/neo4j/gis/spatial/LayersTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -365,7 +365,7 @@ public void testIndexAccessAfterBulkInsertion() {
365365
coordinateNodes.add(node);
366366
}
367367
layer.addAll(tx, coordinateNodes);
368-
layer.close(tx);
368+
layer.finalizeTransaction(tx);
369369
tx.commit();
370370
}
371371

0 commit comments

Comments
 (0)