Make neo4j-graph-algorithms work with 3.2 and all the Kernel-API changes

This creates a new "Kernel" class that wraps the API to make it behave like the old one
Going forward we should rather change the code to not rely on Cursors during the loading
but just use primitive values and iterators.

Something that is also worse now in the new Kernel API is the massive use of (checked) Exceptions
which is definitely not performance improving.

Author: jexp

algo/src/main/java/org/neo4j/graphalgo/exporter/SCCTarjanExporter.java

@@ -9,7 +9,7 @@
 import org.neo4j.kernel.api.exceptions.EntityNotFoundException;
 import org.neo4j.kernel.api.exceptions.InvalidTransactionTypeKernelException;
 import org.neo4j.kernel.api.exceptions.legacyindex.AutoIndexingKernelException;
-import org.neo4j.kernel.api.exceptions.schema.ConstraintValidationKernelException;
+import org.neo4j.kernel.api.exceptions.schema.ConstraintValidationException;
 import org.neo4j.kernel.api.exceptions.schema.IllegalTokenNameException;
 import org.neo4j.kernel.api.properties.DefinedProperty;
 import org.neo4j.kernel.internal.GraphDatabaseAPI;
@@ -36,13 +36,7 @@ public SCCTarjanExporter withIdMapping(IdMapping idMapping) {
     }
 
     public SCCTarjanExporter withWriteProperty(String writeProperty) {
-        writeInTransaction(write -> {
-            try {
-                this.writePropertyId = write.propertyKeyGetOrCreateForName(writeProperty);
-            } catch (IllegalTokenNameException e) {
-                throw new RuntimeException(e);
-            }
-        });
+        this.writePropertyId = getOrCreatePropertyId(writeProperty);
         return this;
     }
 
@@ -57,7 +51,7 @@ public void write(ObjectArrayList<IntSet> data) {
                         write.nodeSetProperty(idMapping.toOriginalNodeId(iCursor.value),
                                 property);
                     } catch (EntityNotFoundException
-                            | ConstraintValidationKernelException
+                            | ConstraintValidationException
                             | InvalidTransactionTypeKernelException
                             | AutoIndexingKernelException e) {
                         throw new RuntimeException(e);

algo/src/main/java/org/neo4j/graphalgo/impl/BetweennessCentrality.java

@@ -6,6 +6,12 @@
 import org.neo4j.graphalgo.core.utils.TerminationFlag;
 import org.neo4j.graphalgo.core.utils.container.Path;
 import org.neo4j.graphdb.Direction;
+import org.neo4j.kernel.api.exceptions.EntityNotFoundException;
+import org.neo4j.kernel.api.exceptions.InvalidTransactionTypeKernelException;
+import org.neo4j.kernel.api.exceptions.legacyindex.AutoIndexingKernelException;
+import org.neo4j.kernel.api.exceptions.schema.ConstraintValidationException;
+import org.neo4j.kernel.api.properties.Property;
+import org.neo4j.kernel.internal.GraphDatabaseAPI;
 
 import java.util.Arrays;
 import java.util.stream.IntStream;

algo/src/main/java/org/neo4j/graphalgo/impl/ShortestPathDijkstra.java

@@ -10,7 +10,7 @@
 import org.neo4j.kernel.api.exceptions.EntityNotFoundException;
 import org.neo4j.kernel.api.exceptions.InvalidTransactionTypeKernelException;
 import org.neo4j.kernel.api.exceptions.legacyindex.AutoIndexingKernelException;
-import org.neo4j.kernel.api.exceptions.schema.ConstraintValidationKernelException;
+import org.neo4j.kernel.api.exceptions.schema.ConstraintValidationException;
 import org.neo4j.kernel.api.properties.DefinedProperty;
 import org.neo4j.kernel.impl.util.collection.SimpleBitSet;
 import org.neo4j.kernel.internal.GraphDatabaseAPI;
@@ -199,7 +199,7 @@ public void write(IntArrayDeque data) {
                     try {
                         writeOp.nodeSetProperty(idMapping.toOriginalNodeId(node),
                                 DefinedProperty.numberProperty(propertyId, distance++));
-                    } catch (EntityNotFoundException | ConstraintValidationKernelException | InvalidTransactionTypeKernelException | AutoIndexingKernelException e) {
+                    } catch (EntityNotFoundException | ConstraintValidationException | InvalidTransactionTypeKernelException | AutoIndexingKernelException e) {
                         throw new RuntimeException(e);
                     }
                 }

core/src/main/java/org/neo4j/graphalgo/api/GraphFactory.java

@@ -2,8 +2,8 @@
 
 import org.neo4j.graphalgo.core.utils.ProgressLogger;
 import org.neo4j.graphalgo.core.utils.ProgressLoggerAdapter;
+import org.neo4j.graphalgo.core.Kernel;
 import org.neo4j.graphdb.Transaction;
-import org.neo4j.kernel.api.ReadOperations;
 import org.neo4j.kernel.api.Statement;
 import org.neo4j.kernel.impl.core.ThreadToStatementContextBridge;
 import org.neo4j.kernel.internal.GraphDatabaseAPI;
@@ -46,10 +46,10 @@ public void setLog(Log log) {
      *
      * @param block the consumer
      */
-    protected final void withReadOps(Consumer<ReadOperations> block) {
+    protected final void withReadOps(Consumer<Kernel> block) {
         try (final Transaction tx = api.beginTx();
              Statement statement = contextBridge.get()) {
-            block.accept(statement.readOperations());
+            block.accept(new Kernel(statement));
             tx.success();
         }
     }