This is an automated email from the ASF dual-hosted git repository. szetszwo pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/ratis.git
commit bc5416828718bd77bd1a31d9dee2e1c4e768982d Author: William Song <[email protected]> AuthorDate: Thu Jan 23 00:38:00 2025 +0800 RATIS-2240. updatePurgeIndex doesn't need to hold RaftLogBase.writeLock (#1214) --- .../src/main/java/org/apache/ratis/server/raftlog/RaftLogBase.java | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) 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 index b37d40c28..710dede48 100644 --- 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 abstract class RaftLogBase implements RaftLog { } protected void updatePurgeIndex(Long purged) { - try (AutoCloseableLock writeLock = writeLock()) { - if (purged != null) { - purgeIndex.updateToMax(purged, infoIndexChange); - } + if (purged != null) { + purgeIndex.updateToMax(purged, infoIndexChange); } }
