less synchronization

terrier-org · Jan 8, 2025 · 047a087 · 047a087
1 parent 44580f7
commit 047a087
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/...s/concurrent/src/main/java/org/terrier/structures/concurrent/ConcurrentDocumentIndex.java b/...s/concurrent/src/main/java/org/terrier/structures/concurrent/ConcurrentDocumentIndex.java
@@ -47,9 +47,8 @@ public DocumentIndexEntry getDocumentEntry(int docid) throws IOException {
 	}
 
 	public int getDocumentLength(int docid) throws IOException {
-		synchronized (parent) {
-			return parent.getDocumentLength(docid);
-		}		
+		// we dont synchronize on getDocumentLength() - we assume this is always in memory and reentrant
+		return parent.getDocumentLength(docid);	
 	}
 
 	public int getNumberOfDocuments() {