X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?p=controller.git;a=blobdiff_plain;f=opendaylight%2Fmd-sal%2Fsal-akka-raft%2Fsrc%2Fmain%2Fjava%2Forg%2Fopendaylight%2Fcontroller%2Fcluster%2Fraft%2Fclient%2Fmessages%2FGetSnapshotReply.java;h=36c534654cabb80469c5035522805c351f354f37;hp=b0c36ace2b4174ae5f675a2c7200e7df2b435943;hb=b4bf55727093657662d8c16a50fa85f87978a586;hpb=9d4ff4c4045fdef38e3940d336d2825df29c4d65 diff --git a/opendaylight/md-sal/sal-akka-raft/src/main/java/org/opendaylight/controller/cluster/raft/client/messages/GetSnapshotReply.java b/opendaylight/md-sal/sal-akka-raft/src/main/java/org/opendaylight/controller/cluster/raft/client/messages/GetSnapshotReply.java index b0c36ace2b..36c534654c 100644 --- a/opendaylight/md-sal/sal-akka-raft/src/main/java/org/opendaylight/controller/cluster/raft/client/messages/GetSnapshotReply.java +++ b/opendaylight/md-sal/sal-akka-raft/src/main/java/org/opendaylight/controller/cluster/raft/client/messages/GetSnapshotReply.java @@ -7,9 +7,10 @@ */ package org.opendaylight.controller.cluster.raft.client.messages; -import com.google.common.base.Preconditions; +import static java.util.Objects.requireNonNull; + import edu.umd.cs.findbugs.annotations.SuppressFBWarnings; -import javax.annotation.Nonnull; +import org.eclipse.jdt.annotation.NonNull; import org.opendaylight.controller.cluster.raft.persisted.Snapshot; /** @@ -21,21 +22,19 @@ public class GetSnapshotReply { private final String id; private final Snapshot snapshot; - public GetSnapshotReply(@Nonnull String id, @Nonnull Snapshot snapshot) { - this.id = Preconditions.checkNotNull(id); - this.snapshot = Preconditions.checkNotNull(snapshot); + public GetSnapshotReply(@NonNull String id, @NonNull Snapshot snapshot) { + this.id = requireNonNull(id); + this.snapshot = requireNonNull(snapshot); } - @Nonnull - public String getId() { + public @NonNull String getId() { return id; } @SuppressFBWarnings(value = "EI_EXPOSE_REP", justification = "Exposes a mutable object stored in a field but " + "this is OK since this class is merely a DTO and does not process the byte[] internally. " + "Also it would be inefficient to create a return copy as the byte[] could be large.") - @Nonnull - public Snapshot getSnapshot() { + public @NonNull Snapshot getSnapshot() { return snapshot; }