Test correctness of reading property chains with concurrent record reuse

neo4j · Jun 30, 2016 · 4e95a46 · 4e95a46
1 parent 5d91e08
commit 4e95a46
Show file tree

Hide file tree

Showing 2 changed files with 294 additions and 325 deletions.
diff --git a/community/kernel/src/main/java/org/neo4j/kernel/impl/api/OperationsFacade.java b/community/kernel/src/main/java/org/neo4j/kernel/impl/api/OperationsFacade.java
@@ -58,9 +58,9 @@
 import org.neo4j.kernel.api.exceptions.schema.IllegalTokenNameException;
 import org.neo4j.kernel.api.exceptions.schema.IndexBrokenKernelException;
 import org.neo4j.kernel.api.exceptions.schema.IndexSchemaRuleNotFoundException;
+import org.neo4j.kernel.api.exceptions.schema.ProcedureConstraintViolation;
 import org.neo4j.kernel.api.exceptions.schema.SchemaRuleNotFoundException;
 import org.neo4j.kernel.api.exceptions.schema.TooManyLabelsException;
-import org.neo4j.kernel.api.exceptions.schema.ProcedureConstraintViolation;
 import org.neo4j.kernel.api.index.IndexDescriptor;
 import org.neo4j.kernel.api.index.InternalIndexState;
 import org.neo4j.kernel.api.procedures.ProcedureDescriptor;
@@ -401,6 +401,10 @@ public Object relationshipGetProperty( long relationshipId, int propertyKeyId )
         {
             return relationship.get().getProperty( propertyKeyId );
         }
+        finally
+        {
+            statement.assertOpen();
+        }
     }
 
     @Override
@@ -433,6 +437,10 @@ public PrimitiveIntIterator nodeGetPropertyKeys( long nodeId ) throws EntityNotF
         {
             return node.get().getPropertyKeys();
         }
+        finally
+        {
+            statement.assertOpen();
+        }
     }
 
     @Override
@@ -443,6 +451,10 @@ public PrimitiveIntIterator relationshipGetPropertyKeys( long relationshipId ) t
         {
             return relationship.get().getPropertyKeys();
         }
+        finally
+        {
+            statement.assertOpen();
+        }
     }
 
     @Override