refactor: use parameter instead of object

Instead of holding the object reference in the lambda use the commitIndex parameter

(cherry picked from commit a88be3e)

atomix/cluster/src/main/java/io/atomix/raft/roles/LeaderRole.java

@@ -601,8 +601,7 @@ private void replicate(final IndexedRaftLogEntry indexed, final AppendListener a
     raft.checkThread();
     final var appendEntriesFuture = appender.appendEntries(indexed.index());
 
-    final boolean applicationEntryWasCommitted = indexed.isApplicationEntry();
-    if (applicationEntryWasCommitted) {
+    if (indexed.isApplicationEntry()) {
       // We have some services which are waiting for the application records, especially position
       // to be committed. This is our glue code to notify them, instead of
       // passing the complete object (IndexedRaftLogEntry) threw the listeners and
@@ -632,11 +631,11 @@ private void replicate(final IndexedRaftLogEntry indexed, final AppendListener a
           // up to date with the latest entries, so it can handle configuration and initial
           // entries properly on fail over
           if (commitError == null) {
-            appendListener.onCommit(indexed.index());
+            appendListener.onCommit(commitIndex);
           } else {
-            appendListener.onCommitError(indexed.index(), commitError);
+            appendListener.onCommitError(commitIndex, commitError);
             // replicating the entry will be retried on the next append request
-            log.error("Failed to replicate entry: {}", indexed, commitError);
+            log.error("Failed to replicate entry: {}", commitIndex, commitError);
           }
         },
         raft.getThreadContext());