X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?p=controller.git;a=blobdiff_plain;f=opendaylight%2Fmd-sal%2Fsal-clustering-commons%2Fsrc%2Fmain%2Fjava%2Forg%2Fopendaylight%2Fcontroller%2Fcluster%2Fmessaging%2FMessageAssembler.java;h=45e3a8e828fef4ba58c321b8437902e9aaaa5b55;hp=328ff0c44a507c9301cbe89f16a18b3ae68668fb;hb=b5cb353e3553a39f576c284119af75ffa5ea66a9;hpb=b712eb01354ddb5878008e2a2e8f03fb19b92555 diff --git a/opendaylight/md-sal/sal-clustering-commons/src/main/java/org/opendaylight/controller/cluster/messaging/MessageAssembler.java b/opendaylight/md-sal/sal-clustering-commons/src/main/java/org/opendaylight/controller/cluster/messaging/MessageAssembler.java index 328ff0c44a..45e3a8e828 100644 --- a/opendaylight/md-sal/sal-clustering-commons/src/main/java/org/opendaylight/controller/cluster/messaging/MessageAssembler.java +++ b/opendaylight/md-sal/sal-clustering-commons/src/main/java/org/opendaylight/controller/cluster/messaging/MessageAssembler.java @@ -40,7 +40,7 @@ public class MessageAssembler implements AutoCloseable { private final BiConsumer assembledMessageCallback; private final String logContext; - private MessageAssembler(Builder builder) { + private MessageAssembler(final Builder builder) { this.fileBackedStreamFactory = Preconditions.checkNotNull(builder.fileBackedStreamFactory, "FiledBackedStreamFactory cannot be null"); this.assembledMessageCallback = Preconditions.checkNotNull(builder.assembledMessageCallback, @@ -69,7 +69,7 @@ public class MessageAssembler implements AutoCloseable { * @param message the message to check * @return true if handled, false otherwise */ - public static boolean isHandledMessage(Object message) { + public static boolean isHandledMessage(final Object message) { return message instanceof MessageSlice || message instanceof AbortSlicing; } @@ -144,7 +144,7 @@ public class MessageAssembler implements AutoCloseable { messageSlice.getSliceIndex()), true); } - private void processMessageSliceForState(final MessageSlice messageSlice, AssembledMessageState state, + private void processMessageSliceForState(final MessageSlice messageSlice, final AssembledMessageState state, final ActorRef sendTo) { final Identifier identifier = messageSlice.getIdentifier(); final ActorRef replyTo = messageSlice.getReplyTo(); @@ -177,7 +177,7 @@ public class MessageAssembler implements AutoCloseable { } } - private Object reAssembleMessage(final AssembledMessageState state) throws MessageSliceException { + private static Object reAssembleMessage(final AssembledMessageState state) throws MessageSliceException { try { final ByteSource assembledBytes = state.getAssembledBytes(); try (ObjectInputStream in = new ObjectInputStream(assembledBytes.openStream())) { @@ -190,7 +190,7 @@ public class MessageAssembler implements AutoCloseable { } } - private void onAbortSlicing(AbortSlicing message) { + private void onAbortSlicing(final AbortSlicing message) { removeState(message.getIdentifier()); } @@ -199,7 +199,7 @@ public class MessageAssembler implements AutoCloseable { stateCache.invalidate(identifier); } - private void stateRemoved(RemovalNotification notification) { + private void stateRemoved(final RemovalNotification notification) { if (notification.wasEvicted()) { LOG.warn("{}: AssembledMessageState for {} was expired from the cache", logContext, notification.getKey()); } else { @@ -211,7 +211,7 @@ public class MessageAssembler implements AutoCloseable { } @VisibleForTesting - boolean hasState(Identifier forIdentifier) { + boolean hasState(final Identifier forIdentifier) { boolean exists = stateCache.getIfPresent(forIdentifier) != null; stateCache.cleanUp(); return exists;