RATIS-2240. updatePurgeIndex doesn't need to hold RaftLogBase.writeLock (#1214)

SzyWilliam · web-flow · commit 345641fb4ce4 · 2025-01-22T08:38:00.000-08:00
diff --git a/ratis-server/src/main/java/org/apache/ratis/server/raftlog/RaftLogBase.java b/ratis-server/src/main/java/org/apache/ratis/server/raftlog/RaftLogBase.java
@@ -144,10 +144,8 @@ public boolean updateCommitIndex(long majorityIndex, long currentTerm, boolean i
   }
 
   protected void updatePurgeIndex(Long purged) {
-    try (AutoCloseableLock writeLock = writeLock()) {
-      if (purged != null) {
-        purgeIndex.updateToMax(purged, infoIndexChange);
-      }
+    if (purged != null) {
+      purgeIndex.updateToMax(purged, infoIndexChange);
     }
   }
 

Original file line number	Diff line number	Diff line change
`@@ -144,10 +144,8 @@ public boolean updateCommitIndex(long majorityIndex, long currentTerm, boolean i`
`144`	`144`	`}`
`145`	`145`
`146`	`146`	`protected void updatePurgeIndex(Long purged) {`
`147`		`- try (AutoCloseableLock writeLock = writeLock()) {`
`148`		`- if (purged != null) {`
`149`		`- purgeIndex.updateToMax(purged, infoIndexChange);`
`150`		`- }`
	`147`	`+ if (purged != null) {`
	`148`	`+ purgeIndex.updateToMax(purged, infoIndexChange);`
`151`	`149`	`}`
`152`	`150`	`}`
`153`	`151`