X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?a=blobdiff_plain;f=opendaylight%2Fmd-sal%2Fsal-akka-raft%2Fsrc%2Fmain%2Fjava%2Forg%2Fopendaylight%2Fcontroller%2Fcluster%2Fraft%2FReplicatedLogImpl.java;h=c53cad68c5def14c0dae3718ffc01144235db5fe;hb=7daddd72031b33ed686abe18a0813e41263aac8d;hp=45b0b3898e20cc3b3a5acbe523944bbfc0ee6c6a;hpb=b4bf55727093657662d8c16a50fa85f87978a586;p=controller.git diff --git a/opendaylight/md-sal/sal-akka-raft/src/main/java/org/opendaylight/controller/cluster/raft/ReplicatedLogImpl.java b/opendaylight/md-sal/sal-akka-raft/src/main/java/org/opendaylight/controller/cluster/raft/ReplicatedLogImpl.java index 45b0b3898e..c53cad68c5 100644 --- a/opendaylight/md-sal/sal-akka-raft/src/main/java/org/opendaylight/controller/cluster/raft/ReplicatedLogImpl.java +++ b/opendaylight/md-sal/sal-akka-raft/src/main/java/org/opendaylight/controller/cluster/raft/ReplicatedLogImpl.java @@ -101,22 +101,35 @@ final class ReplicatedLogImpl extends AbstractReplicatedLogImpl { return false; } - Procedure persistCallback = persistedLogEntry -> { - context.getLogger().debug("{}: persist complete {}", context.getId(), persistedLogEntry); - - dataSizeSinceLastSnapshot += persistedLogEntry.size(); - - if (callback != null) { - callback.apply(persistedLogEntry); - } - }; - if (doAsync) { - context.getPersistenceProvider().persistAsync(replicatedLogEntry, persistCallback); + context.getPersistenceProvider().persistAsync(replicatedLogEntry, + entry -> persistCallback(entry, callback)); } else { - context.getPersistenceProvider().persist(replicatedLogEntry, persistCallback); + context.getPersistenceProvider().persist(replicatedLogEntry, entry -> syncPersistCallback(entry, callback)); } return true; } + + private void persistCallback(final ReplicatedLogEntry persistedLogEntry, + final Procedure callback) { + context.getExecutor().execute(() -> syncPersistCallback(persistedLogEntry, callback)); + } + + @SuppressWarnings("checkstyle:illegalCatch") + private void syncPersistCallback(final ReplicatedLogEntry persistedLogEntry, + final Procedure callback) { + context.getLogger().debug("{}: persist complete {}", context.getId(), persistedLogEntry); + + dataSizeSinceLastSnapshot += persistedLogEntry.size(); + + if (callback != null) { + try { + callback.apply(persistedLogEntry); + } catch (Exception e) { + context.getLogger().error("{}: persist callback failed", context.getId(), e); + throw new IllegalStateException("Persist callback failed", e); + } + } + } }