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

SzyWilliam · OneSizeFitsQuorum · commit 6708ab940628 · 2025-04-28T15:25:29.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
@@ -141,10 +141,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
`@@ -141,10 +141,8 @@ public boolean updateCommitIndex(long majorityIndex, long currentTerm, boolean i`
`141`	`141`	`}`
`142`	`142`
`143`	`143`	`protected void updatePurgeIndex(Long purged) {`
`144`		`- try (AutoCloseableLock writeLock = writeLock()) {`
`145`		`- if (purged != null) {`
`146`		`- purgeIndex.updateToMax(purged, infoIndexChange);`
`147`		`- }`
	`144`	`+ if (purged != null) {`
	`145`	`+ purgeIndex.updateToMax(purged, infoIndexChange);`
`148`	`146`	`}`
`149`	`147`	`}`
`150`	`148`