More cds-access-api cleanup 64/103064/1
authorRobert Varga <robert.varga@pantheon.tech>
Thu, 3 Nov 2022 13:12:34 +0000 (14:12 +0100)
committerRobert Varga <robert.varga@pantheon.tech>
Thu, 3 Nov 2022 13:12:34 +0000 (14:12 +0100)
Fixup previous patch and sprinkle more @Serial annotations.

Change-Id: I74414861197bb417dadc445ebc3fc2703a97ea4d
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
58 files changed:
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/commands/AbstractLocalHistoryRequestProxy.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/commands/AbstractLocalTransactionRequest.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/commands/AbstractReadPathTransactionRequestProxyV1.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/commands/AbstractReadTransactionRequestProxyV1.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/commands/AbstractTransactionRequestProxy.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/commands/AbstractTransactionSuccessProxy.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/commands/ConnectClientRequest.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/commands/ConnectClientSuccess.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/commands/ConnectClientSuccessProxyV1.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/commands/CreateLocalHistoryRequestProxyV1.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/commands/DestroyLocalHistoryRequestProxyV1.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/commands/ExistsTransactionRequestProxyV1.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/commands/ExistsTransactionSuccessProxyV1.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/commands/IncrementTransactionSequenceRequest.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/commands/IncrementTransactionSequenceSuccessProxyV1.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/commands/LocalHistoryFailureProxyV1.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/commands/LocalHistorySuccess.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/commands/LocalHistorySuccessProxyV1.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/commands/ModifyTransactionRequestProxyV1.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/commands/ModifyTransactionSuccessProxyV1.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/commands/PurgeLocalHistoryRequestProxyV1.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/commands/ReadTransactionRequestProxyV1.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/commands/ReadTransactionSuccessProxyV1.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/commands/SkipTransactionsResponseProxyV1.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/commands/TransactionAbortRequestProxyV1.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/commands/TransactionAbortSuccess.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/commands/TransactionAbortSuccessProxyV1.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/commands/TransactionCanCommitSuccess.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/commands/TransactionCanCommitSuccessProxyV1.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/commands/TransactionCommitSuccess.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/commands/TransactionCommitSuccessProxyV1.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/commands/TransactionDoCommitRequestProxyV1.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/commands/TransactionFailureProxyV1.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/commands/TransactionPreCommitRequestProxyV1.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/commands/TransactionPreCommitSuccess.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/commands/TransactionPreCommitSuccessProxyV1.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/commands/TransactionPurgeRequestProxyV1.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/commands/TransactionPurgeResponse.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/commands/TransactionPurgeResponseProxyV1.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/concepts/AbstractEnvelopeProxy.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/concepts/AbstractMessageProxy.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/concepts/AbstractResponseEnvelopeProxy.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/concepts/AbstractResponseProxy.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/concepts/ClientIdentifier.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/concepts/Envelope.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/concepts/FailureEnvelope.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/concepts/FailureEnvelopeProxy.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/concepts/FrontendIdentifier.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/concepts/FrontendType.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/concepts/LocalHistoryIdentifier.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/concepts/MemberName.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/concepts/Message.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/concepts/RequestEnvelope.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/concepts/RequestEnvelopeProxy.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/concepts/ResponseEnvelope.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/concepts/SuccessEnvelope.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/concepts/SuccessEnvelopeProxy.java
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/concepts/TransactionIdentifier.java

index f6ba2e7970b1435a741cb5b9c41fa6c3cf5728d6..cd02a20099597500687525db4ec5816679527714 100644 (file)
@@ -9,6 +9,7 @@ package org.opendaylight.controller.cluster.access.commands;
 
 import java.io.DataInput;
 import java.io.IOException;
+import java.io.Serial;
 import org.opendaylight.controller.cluster.access.concepts.AbstractRequestProxy;
 import org.opendaylight.controller.cluster.access.concepts.LocalHistoryIdentifier;
 
@@ -21,6 +22,7 @@ import org.opendaylight.controller.cluster.access.concepts.LocalHistoryIdentifie
  */
 abstract class AbstractLocalHistoryRequestProxy<T extends LocalHistoryRequest<T>>
         extends AbstractRequestProxy<LocalHistoryIdentifier, T> {
+    @Serial
     private static final long serialVersionUID = 1L;
 
     protected AbstractLocalHistoryRequestProxy() {
index efc0e856b20c4d7ed8ebeb7f49dc9460bfa28767..f31e4088890e6a0a0486caf32dd3325b481ffd91 100644 (file)
@@ -8,6 +8,7 @@
 package org.opendaylight.controller.cluster.access.commands;
 
 import akka.actor.ActorRef;
+import java.io.Serial;
 import org.opendaylight.controller.cluster.access.ABIVersion;
 import org.opendaylight.controller.cluster.access.concepts.Request;
 import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier;
@@ -22,6 +23,7 @@ import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier
  */
 public abstract class AbstractLocalTransactionRequest<T extends AbstractLocalTransactionRequest<T>>
         extends TransactionRequest<T> {
+    @Serial
     private static final long serialVersionUID = 1L;
 
     AbstractLocalTransactionRequest(final TransactionIdentifier identifier, final long sequence,
index 2cd4ec9e73340ecbcb60f96b2c7bd4370c884e8b..e139920d6f2f064ea9552a2cd7a35b118e754fea 100644 (file)
@@ -11,6 +11,7 @@ import akka.actor.ActorRef;
 import java.io.IOException;
 import java.io.ObjectInput;
 import java.io.ObjectOutput;
+import java.io.Serial;
 import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier;
 import org.opendaylight.yangtools.yang.data.api.YangInstanceIdentifier;
 import org.opendaylight.yangtools.yang.data.codec.binfmt.NormalizedNodeDataInput;
@@ -27,6 +28,7 @@ import org.opendaylight.yangtools.yang.data.codec.binfmt.NormalizedNodeStreamVer
  */
 abstract class AbstractReadPathTransactionRequestProxyV1<T extends AbstractReadPathTransactionRequest<T>>
         extends AbstractReadTransactionRequestProxyV1<T> {
+    @Serial
     private static final long serialVersionUID = 1L;
 
     private YangInstanceIdentifier path;
index 88820bd1f53e7d5e06a7fb6cec1d759904c25c6c..66bca1d7465e15e2da21ef669fb433f25420baff 100644 (file)
@@ -11,6 +11,7 @@ import akka.actor.ActorRef;
 import java.io.IOException;
 import java.io.ObjectInput;
 import java.io.ObjectOutput;
+import java.io.Serial;
 import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier;
 
 /**
@@ -23,7 +24,9 @@ import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier
  */
 abstract class AbstractReadTransactionRequestProxyV1<T extends AbstractReadTransactionRequest<T>>
         extends AbstractTransactionRequestProxy<T> {
+    @Serial
     private static final long serialVersionUID = 1L;
+
     private boolean snapshotOnly;
 
     protected AbstractReadTransactionRequestProxyV1() {
index 437d281e1848f93eb64db58a7c63389117069340..b37227056a131d76331c6d3d30502a527e21eb16 100644 (file)
@@ -9,6 +9,7 @@ package org.opendaylight.controller.cluster.access.commands;
 
 import java.io.DataInput;
 import java.io.IOException;
+import java.io.Serial;
 import org.opendaylight.controller.cluster.access.concepts.AbstractRequestProxy;
 import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier;
 
@@ -21,9 +22,10 @@ import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier
  */
 abstract class AbstractTransactionRequestProxy<T extends TransactionRequest<T>>
         extends AbstractRequestProxy<TransactionIdentifier, T> {
+    @Serial
     private static final long serialVersionUID = 1L;
 
-    protected AbstractTransactionRequestProxy() {
+    AbstractTransactionRequestProxy() {
         // For Externalizable
     }
 
index a1284b703ff2c7d5d55f99299b4afd9513110b3f..187ad71c35185c93eac612de9174f86f05c13363 100644 (file)
@@ -9,6 +9,7 @@ package org.opendaylight.controller.cluster.access.commands;
 
 import java.io.DataInput;
 import java.io.IOException;
+import java.io.Serial;
 import org.opendaylight.controller.cluster.access.concepts.AbstractSuccessProxy;
 import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier;
 
@@ -21,9 +22,10 @@ import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier
  */
 abstract class AbstractTransactionSuccessProxy<T extends TransactionSuccess<T>>
         extends AbstractSuccessProxy<TransactionIdentifier, T> {
+    @Serial
     private static final long serialVersionUID = 1L;
 
-    protected AbstractTransactionSuccessProxy() {
+    AbstractTransactionSuccessProxy() {
         // For Externalizable
     }
 
index 1b937521efb836dfe9f7ef810cd1f46d31cd14d5..12534e037def93dbcd50a40655c3904eb076dc86 100644 (file)
@@ -11,6 +11,7 @@ import static java.util.Objects.requireNonNull;
 
 import akka.actor.ActorRef;
 import com.google.common.base.MoreObjects.ToStringHelper;
+import java.io.Serial;
 import org.opendaylight.controller.cluster.access.ABIVersion;
 import org.opendaylight.controller.cluster.access.concepts.AbstractRequestProxy;
 import org.opendaylight.controller.cluster.access.concepts.ClientIdentifier;
@@ -27,6 +28,7 @@ import org.opendaylight.controller.cluster.access.concepts.RequestException;
  * It also includes request stream sequencing information.
  */
 public final class ConnectClientRequest extends Request<ClientIdentifier, ConnectClientRequest> {
+    @Serial
     private static final long serialVersionUID = 1L;
 
     private final ABIVersion minVersion;
index ae4426515d4d1bc8a6aa8d2f3547a70b6975d518..d87a486f7917ff512176172eee67b7539dd0891d 100644 (file)
@@ -15,6 +15,7 @@ import akka.actor.ActorSelection;
 import com.google.common.base.MoreObjects.ToStringHelper;
 import com.google.common.collect.ImmutableList;
 import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
+import java.io.Serial;
 import java.util.List;
 import java.util.Optional;
 import org.eclipse.jdt.annotation.NonNull;
@@ -29,6 +30,7 @@ import org.opendaylight.yangtools.yang.data.tree.api.ReadOnlyDataTree;
  * fail, the client can try accessing the provided alternates.
  */
 public final class ConnectClientSuccess extends RequestSuccess<ClientIdentifier, ConnectClientSuccess> {
+    @Serial
     private static final long serialVersionUID = 1L;
 
     private final @NonNull ImmutableList<ActorSelection> alternates;
index 8dd40ac2c43d9c6dc0846bb3c2aff66ea2fdbc5d..ed04f031974c6915b323ac73ad009985d3b7aab0 100644 (file)
@@ -15,6 +15,7 @@ import java.io.DataInput;
 import java.io.IOException;
 import java.io.ObjectInput;
 import java.io.ObjectOutput;
+import java.io.Serial;
 import java.util.ArrayList;
 import java.util.List;
 import org.opendaylight.controller.cluster.access.concepts.AbstractSuccessProxy;
@@ -27,6 +28,7 @@ import org.opendaylight.controller.cluster.access.concepts.ClientIdentifier;
  * @author Robert Varga
  */
 final class ConnectClientSuccessProxyV1 extends AbstractSuccessProxy<ClientIdentifier, ConnectClientSuccess> {
+    @Serial
     private static final long serialVersionUID = 1L;
 
     private List<ActorSelection> alternates;
@@ -42,9 +44,9 @@ final class ConnectClientSuccessProxyV1 extends AbstractSuccessProxy<ClientIdent
 
     ConnectClientSuccessProxyV1(final ConnectClientSuccess success) {
         super(success);
-        this.alternates = success.getAlternates();
-        this.backend = success.getBackend();
-        this.maxMessages = success.getMaxMessages();
+        alternates = success.getAlternates();
+        backend = success.getBackend();
+        maxMessages = success.getMaxMessages();
         // We are ignoring the DataTree, it is not serializable anyway
     }
 
index b61c9f5309d5617e327e1f3903de41f7016c65fc..ff3d09844ee9ce0a32b636fb1090942e14435fe9 100644 (file)
@@ -8,6 +8,7 @@
 package org.opendaylight.controller.cluster.access.commands;
 
 import akka.actor.ActorRef;
+import java.io.Serial;
 import org.opendaylight.controller.cluster.access.concepts.LocalHistoryIdentifier;
 
 /**
@@ -17,6 +18,7 @@ import org.opendaylight.controller.cluster.access.concepts.LocalHistoryIdentifie
  * @author Robert Varga
  */
 final class CreateLocalHistoryRequestProxyV1 extends AbstractLocalHistoryRequestProxy<CreateLocalHistoryRequest> {
+    @Serial
     private static final long serialVersionUID = 1L;
 
     // checkstyle flags the public modifier as redundant however it is explicitly needed for Java serialization to
index 0ebd69054d9ea424bd9ca6730243c8e4dad3bab3..d5eb96e9700d83efcee7a513b4e976314c3abf59 100644 (file)
@@ -8,6 +8,7 @@
 package org.opendaylight.controller.cluster.access.commands;
 
 import akka.actor.ActorRef;
+import java.io.Serial;
 import org.opendaylight.controller.cluster.access.concepts.LocalHistoryIdentifier;
 
 /**
@@ -17,6 +18,7 @@ import org.opendaylight.controller.cluster.access.concepts.LocalHistoryIdentifie
  * @author Robert Varga
  */
 final class DestroyLocalHistoryRequestProxyV1 extends AbstractLocalHistoryRequestProxy<DestroyLocalHistoryRequest> {
+    @Serial
     private static final long serialVersionUID = 1L;
 
     // checkstyle flags the public modifier as redundant however it is explicitly needed for Java serialization to
index 24299471558bb3be08144ce12ad3831d992414f3..a899da944cae30cdc8303fb41022cc6ea6d70f6e 100644 (file)
@@ -8,6 +8,7 @@
 package org.opendaylight.controller.cluster.access.commands;
 
 import akka.actor.ActorRef;
+import java.io.Serial;
 import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier;
 import org.opendaylight.yangtools.yang.data.api.YangInstanceIdentifier;
 
@@ -19,6 +20,7 @@ import org.opendaylight.yangtools.yang.data.api.YangInstanceIdentifier;
  */
 final class ExistsTransactionRequestProxyV1 extends
         AbstractReadPathTransactionRequestProxyV1<ExistsTransactionRequest> {
+    @Serial
     private static final long serialVersionUID = 1L;
 
     // checkstyle flags the public modifier as redundant however it is explicitly needed for Java serialization to
index 24de17656ddb48cfaeb24b0e8026f13b40d6d76f..ed66fb2d3d21db7e96bf1f0a41663d21d904507a 100644 (file)
@@ -10,6 +10,7 @@ package org.opendaylight.controller.cluster.access.commands;
 import java.io.IOException;
 import java.io.ObjectInput;
 import java.io.ObjectOutput;
+import java.io.Serial;
 import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier;
 
 /**
@@ -19,7 +20,9 @@ import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier
  * @author Robert Varga
  */
 final class ExistsTransactionSuccessProxyV1 extends AbstractTransactionSuccessProxy<ExistsTransactionSuccess> {
+    @Serial
     private static final long serialVersionUID = 1L;
+
     private boolean exists;
 
     // checkstyle flags the public modifier as redundant however it is explicitly needed for Java serialization to
@@ -31,7 +34,7 @@ final class ExistsTransactionSuccessProxyV1 extends AbstractTransactionSuccessPr
 
     ExistsTransactionSuccessProxyV1(final ExistsTransactionSuccess request) {
         super(request);
-        this.exists = request.getExists();
+        exists = request.getExists();
     }
 
     @Override
index ffc0a68b8912481bfeca882545ff3626469025e0..5e85316c413d75b9040355c4d10b65b334fab496 100644 (file)
@@ -9,6 +9,7 @@ package org.opendaylight.controller.cluster.access.commands;
 
 import akka.actor.ActorRef;
 import com.google.common.base.Preconditions;
+import java.io.Serial;
 import org.opendaylight.controller.cluster.access.ABIVersion;
 import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier;
 
@@ -21,6 +22,7 @@ import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier
  */
 public final class IncrementTransactionSequenceRequest extends
         AbstractReadTransactionRequest<IncrementTransactionSequenceRequest> {
+    @Serial
     private static final long serialVersionUID = 1L;
 
     private final long increment;
index a99faabcffe174652f05075bd4a3cc42311bde94..58c017bb9499b1ab14d65857d2f04b43940e917d 100644 (file)
@@ -7,6 +7,7 @@
  */
 package org.opendaylight.controller.cluster.access.commands;
 
+import java.io.Serial;
 import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier;
 
 /**
@@ -17,6 +18,7 @@ import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier
  */
 final class IncrementTransactionSequenceSuccessProxyV1
         extends AbstractTransactionSuccessProxy<IncrementTransactionSequenceSuccess> {
+    @Serial
     private static final long serialVersionUID = 1L;
 
     // checkstyle flags the public modifier as redundant however it is explicitly needed for Java serialization to
index 0d3a687a41c8b0459ac9b82f1d8080cd7c554397..11ebfa55fbf2f9019c1928eb61aa660b313d257d 100644 (file)
@@ -9,6 +9,7 @@ package org.opendaylight.controller.cluster.access.commands;
 
 import java.io.DataInput;
 import java.io.IOException;
+import java.io.Serial;
 import org.opendaylight.controller.cluster.access.concepts.AbstractRequestFailureProxy;
 import org.opendaylight.controller.cluster.access.concepts.LocalHistoryIdentifier;
 import org.opendaylight.controller.cluster.access.concepts.RequestException;
@@ -21,6 +22,7 @@ import org.opendaylight.controller.cluster.access.concepts.RequestException;
  */
 final class LocalHistoryFailureProxyV1 extends
         AbstractRequestFailureProxy<LocalHistoryIdentifier, LocalHistoryFailure> {
+    @Serial
     private static final long serialVersionUID = 1L;
 
     // checkstyle flags the public modifier as redundant however it is explicitly needed for Java serialization to
index 7480305eb9c444e90ca3c55d80036406b7155aad..99c1de1d619ecd7126822db58f863cc5ab2d0d68 100644 (file)
@@ -7,6 +7,7 @@
  */
 package org.opendaylight.controller.cluster.access.commands;
 
+import java.io.Serial;
 import org.opendaylight.controller.cluster.access.ABIVersion;
 import org.opendaylight.controller.cluster.access.concepts.AbstractSuccessProxy;
 import org.opendaylight.controller.cluster.access.concepts.LocalHistoryIdentifier;
@@ -16,6 +17,7 @@ import org.opendaylight.controller.cluster.access.concepts.RequestSuccess;
  * Success class for {@link RequestSuccess}es involving a specific local history.
  */
 public final class LocalHistorySuccess extends RequestSuccess<LocalHistoryIdentifier, LocalHistorySuccess> {
+    @Serial
     private static final long serialVersionUID = 1L;
 
     public LocalHistorySuccess(final LocalHistoryIdentifier target, final long sequence) {
index 97a7a1c8d6c7e991d4cf629a0ffbc432968c723d..18ea0bd9de0ed48e81d0c6c8e2af477c07ff7134 100644 (file)
@@ -9,6 +9,7 @@ package org.opendaylight.controller.cluster.access.commands;
 
 import java.io.DataInput;
 import java.io.IOException;
+import java.io.Serial;
 import org.opendaylight.controller.cluster.access.concepts.AbstractSuccessProxy;
 import org.opendaylight.controller.cluster.access.concepts.LocalHistoryIdentifier;
 
@@ -18,6 +19,7 @@ import org.opendaylight.controller.cluster.access.concepts.LocalHistoryIdentifie
  * @author Robert Varga
  */
 final class LocalHistorySuccessProxyV1 extends AbstractSuccessProxy<LocalHistoryIdentifier, LocalHistorySuccess> {
+    @Serial
     private static final long serialVersionUID = 1L;
 
     // checkstyle flags the public modifier as redundant however it is explicitly needed for Java serialization to
index 384c889af8ded8aeef6a84b42f5943eb6f53bba8..369aa1515bb7fa9ac488c7c92c436f3529db599e 100644 (file)
@@ -14,6 +14,7 @@ import com.google.common.collect.ImmutableList;
 import java.io.IOException;
 import java.io.ObjectInput;
 import java.io.ObjectOutput;
+import java.io.Serial;
 import java.util.ArrayList;
 import java.util.List;
 import java.util.Optional;
@@ -30,6 +31,7 @@ import org.opendaylight.yangtools.yang.data.impl.schema.ReusableImmutableNormali
  * @author Robert Varga
  */
 final class ModifyTransactionRequestProxyV1 extends AbstractTransactionRequestProxy<ModifyTransactionRequest> {
+    @Serial
     private static final long serialVersionUID = 1L;
 
     private List<TransactionModification> modifications;
@@ -45,9 +47,9 @@ final class ModifyTransactionRequestProxyV1 extends AbstractTransactionRequestPr
 
     ModifyTransactionRequestProxyV1(final ModifyTransactionRequest request) {
         super(request);
-        this.modifications = requireNonNull(request.getModifications());
-        this.protocol = request.getPersistenceProtocol();
-        this.streamVersion = request.getVersion().getStreamVersion();
+        modifications = requireNonNull(request.getModifications());
+        protocol = request.getPersistenceProtocol();
+        streamVersion = request.getVersion().getStreamVersion();
     }
 
     @Override
index 0efff09259f725752670335ce4d744111aef7801..4ead55671cd32d0fb05c2a1cc1912f8ae62be1fe 100644 (file)
@@ -7,6 +7,7 @@
  */
 package org.opendaylight.controller.cluster.access.commands;
 
+import java.io.Serial;
 import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier;
 
 /**
@@ -16,6 +17,7 @@ import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier
  * @author Robert Varga
  */
 final class ModifyTransactionSuccessProxyV1 extends AbstractTransactionSuccessProxy<ModifyTransactionSuccess> {
+    @Serial
     private static final long serialVersionUID = 1L;
 
     // checkstyle flags the public modifier as redundant however it is explicitly needed for Java serialization to
index 11c344f5394dbc21e0869462d3db0686219a3e3f..ff7f4ee4e2f4ab4a618ee74c606c8dbe09762eae 100644 (file)
@@ -8,6 +8,7 @@
 package org.opendaylight.controller.cluster.access.commands;
 
 import akka.actor.ActorRef;
+import java.io.Serial;
 import org.opendaylight.controller.cluster.access.concepts.LocalHistoryIdentifier;
 
 /**
@@ -17,6 +18,7 @@ import org.opendaylight.controller.cluster.access.concepts.LocalHistoryIdentifie
  * @author Robert Varga
  */
 final class PurgeLocalHistoryRequestProxyV1 extends AbstractLocalHistoryRequestProxy<PurgeLocalHistoryRequest> {
+    @Serial
     private static final long serialVersionUID = 1L;
 
     // checkstyle flags the public modifier as redundant however it is explicitly needed for Java serialization to
index a83b6bcaacd2dd4ba7a0eddbcab29a171c9664e4..20d1d308f42bfee4da38d90e99d225228e6e7f14 100644 (file)
@@ -8,6 +8,7 @@
 package org.opendaylight.controller.cluster.access.commands;
 
 import akka.actor.ActorRef;
+import java.io.Serial;
 import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier;
 import org.opendaylight.yangtools.yang.data.api.YangInstanceIdentifier;
 
@@ -18,6 +19,7 @@ import org.opendaylight.yangtools.yang.data.api.YangInstanceIdentifier;
  * @author Robert Varga
  */
 final class ReadTransactionRequestProxyV1 extends AbstractReadPathTransactionRequestProxyV1<ReadTransactionRequest> {
+    @Serial
     private static final long serialVersionUID = 1L;
 
     // checkstyle flags the public modifier as redundant however it is explicitly needed for Java serialization to
index d442f36739281915a101678fc375fde87f51d26f..df462cf0c3abb942c37300895c262216802a3b20 100644 (file)
@@ -10,6 +10,7 @@ package org.opendaylight.controller.cluster.access.commands;
 import java.io.IOException;
 import java.io.ObjectInput;
 import java.io.ObjectOutput;
+import java.io.Serial;
 import java.util.Optional;
 import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier;
 import org.opendaylight.yangtools.yang.data.api.schema.NormalizedNode;
@@ -24,6 +25,7 @@ import org.opendaylight.yangtools.yang.data.codec.binfmt.NormalizedNodeStreamVer
  * @author Robert Varga
  */
 final class ReadTransactionSuccessProxyV1 extends AbstractTransactionSuccessProxy<ReadTransactionSuccess> {
+    @Serial
     private static final long serialVersionUID = 1L;
 
     private Optional<NormalizedNode> data;
@@ -38,8 +40,8 @@ final class ReadTransactionSuccessProxyV1 extends AbstractTransactionSuccessProx
 
     ReadTransactionSuccessProxyV1(final ReadTransactionSuccess request) {
         super(request);
-        this.data = request.getData();
-        this.streamVersion = request.getVersion().getStreamVersion();
+        data = request.getData();
+        streamVersion = request.getVersion().getStreamVersion();
     }
 
     @Override
index 9bc93f9497252dde55c0c3ba5c89805b6250f869..a1bf23de6da7686dc7bf3ad2e3f18140aafc4acc 100644 (file)
@@ -7,6 +7,7 @@
  */
 package org.opendaylight.controller.cluster.access.commands;
 
+import java.io.Serial;
 import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier;
 
 /**
@@ -14,6 +15,7 @@ import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier
  * serialization format.
  */
 final class SkipTransactionsResponseProxyV1 extends AbstractTransactionSuccessProxy<SkipTransactionsResponse> {
+    @Serial
     private static final long serialVersionUID = 1L;
 
     // checkstyle flags the public modifier as redundant however it is explicitly needed for Java serialization to
index 3e67dfe83bff02992afa73dec53daa0c650be490..b4f80f64e41fed7698e5c3a18b35a27f76364c62 100644 (file)
@@ -8,6 +8,7 @@
 package org.opendaylight.controller.cluster.access.commands;
 
 import akka.actor.ActorRef;
+import java.io.Serial;
 import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier;
 
 /**
@@ -17,6 +18,7 @@ import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier
  * @author Robert Varga
  */
 final class TransactionAbortRequestProxyV1 extends AbstractTransactionRequestProxy<TransactionAbortRequest> {
+    @Serial
     private static final long serialVersionUID = 1L;
 
     // checkstyle flags the public modifier as redundant however it is explicitly needed for Java serialization to
index 69c6dddd8f9c49f84801cbb0f938c71b256ec8d3..44de3f69fe17498ba6345661919e0506f8b8b793 100644 (file)
@@ -7,6 +7,7 @@
  */
 package org.opendaylight.controller.cluster.access.commands;
 
+import java.io.Serial;
 import org.opendaylight.controller.cluster.access.ABIVersion;
 import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier;
 
@@ -17,6 +18,7 @@ import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier
  * @author Robert Varga
  */
 public final class TransactionAbortSuccess extends TransactionSuccess<TransactionAbortSuccess> {
+    @Serial
     private static final long serialVersionUID = 1L;
 
     public TransactionAbortSuccess(final TransactionIdentifier identifier, final long sequence) {
index c9de3b97037aa71949b11a146cd32961ade8469b..c04cadc977670d5c6d5dd45ba43661c3c2a936bc 100644 (file)
@@ -7,6 +7,7 @@
  */
 package org.opendaylight.controller.cluster.access.commands;
 
+import java.io.Serial;
 import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier;
 
 /**
@@ -16,6 +17,7 @@ import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier
  * @author Robert Varga
  */
 final class TransactionAbortSuccessProxyV1 extends AbstractTransactionSuccessProxy<TransactionAbortSuccess> {
+    @Serial
     private static final long serialVersionUID = 1L;
 
     // checkstyle flags the public modifier as redundant however it is explicitly needed for Java serialization to
index e6149bd5da226529e746a07e09314fde37759346..720305dc42d4a8dade120359cbcfd4970673b5df 100644 (file)
@@ -7,6 +7,7 @@
  */
 package org.opendaylight.controller.cluster.access.commands;
 
+import java.io.Serial;
 import org.opendaylight.controller.cluster.access.ABIVersion;
 import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier;
 
@@ -17,6 +18,7 @@ import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier
  * @author Robert Varga
  */
 public final class TransactionCanCommitSuccess extends TransactionSuccess<TransactionCanCommitSuccess> {
+    @Serial
     private static final long serialVersionUID = 1L;
 
     public TransactionCanCommitSuccess(final TransactionIdentifier identifier, final long sequence) {
index b41ec2998fd4a5e6b3616e64f90ff5cd526372da..b187ce5247408088dd7bb089662a9ca3cfb97fd3 100644 (file)
@@ -10,6 +10,7 @@ package org.opendaylight.controller.cluster.access.commands;
 import java.io.IOException;
 import java.io.ObjectInput;
 import java.io.ObjectOutput;
+import java.io.Serial;
 import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier;
 
 /**
@@ -19,6 +20,7 @@ import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier
  * @author Robert Varga
  */
 final class TransactionCanCommitSuccessProxyV1 extends AbstractTransactionSuccessProxy<TransactionCanCommitSuccess> {
+    @Serial
     private static final long serialVersionUID = 1L;
 
     // checkstyle flags the public modifier as redundant however it is explicitly needed for Java serialization to
index 6b28244484e3eb739881696429e9119db14dc47e..8ba0b476adddef8862eb0c13cc179382967811bc 100644 (file)
@@ -7,6 +7,7 @@
  */
 package org.opendaylight.controller.cluster.access.commands;
 
+import java.io.Serial;
 import org.opendaylight.controller.cluster.access.ABIVersion;
 import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier;
 
@@ -17,6 +18,7 @@ import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier
  * @author Robert Varga
  */
 public final class TransactionCommitSuccess extends TransactionSuccess<TransactionCommitSuccess> {
+    @Serial
     private static final long serialVersionUID = 1L;
 
     public TransactionCommitSuccess(final TransactionIdentifier identifier, final long sequence) {
index f23da7bac2b98ba719e6fde3543e41571022162a..42a8151db9bdceb2640c7d592b577e6f95f849c4 100644 (file)
@@ -7,6 +7,7 @@
  */
 package org.opendaylight.controller.cluster.access.commands;
 
+import java.io.Serial;
 import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier;
 
 /**
@@ -16,6 +17,7 @@ import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier
  * @author Robert Varga
  */
 final class TransactionCommitSuccessProxyV1 extends AbstractTransactionSuccessProxy<TransactionCommitSuccess> {
+    @Serial
     private static final long serialVersionUID = 1L;
 
     // checkstyle flags the public modifier as redundant however it is explicitly needed for Java serialization to
index fcb63fd0f58f4bef9cd035aab781f2928857984b..c510e3f071e814b85abd6fab73d875209def9b78 100644 (file)
@@ -8,6 +8,7 @@
 package org.opendaylight.controller.cluster.access.commands;
 
 import akka.actor.ActorRef;
+import java.io.Serial;
 import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier;
 
 /**
@@ -17,6 +18,7 @@ import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier
  * @author Robert Varga
  */
 final class TransactionDoCommitRequestProxyV1 extends AbstractTransactionRequestProxy<TransactionDoCommitRequest> {
+    @Serial
     private static final long serialVersionUID = 1L;
 
     // checkstyle flags the public modifier as redundant however it is explicitly needed for Java serialization to
index d3b1dd7cf791389b8402ef87209c0f582554c012..70d1e30e51f3ca85bd44e5488093482c8db8a4cc 100644 (file)
@@ -9,6 +9,7 @@ package org.opendaylight.controller.cluster.access.commands;
 
 import java.io.DataInput;
 import java.io.IOException;
+import java.io.Serial;
 import org.opendaylight.controller.cluster.access.concepts.AbstractRequestFailureProxy;
 import org.opendaylight.controller.cluster.access.concepts.RequestException;
 import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier;
@@ -20,6 +21,7 @@ import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier
  * @author Robert Varga
  */
 final class TransactionFailureProxyV1 extends AbstractRequestFailureProxy<TransactionIdentifier, TransactionFailure> {
+    @Serial
     private static final long serialVersionUID = 1L;
 
     // checkstyle flags the public modifier as redundant however it is explicitly needed for Java serialization to
index bf044e1a1563330239d1fb68d5b0854a8d104ccd..2b56cc352bec54353b20e0953871c8dcbcfa682c 100644 (file)
@@ -8,6 +8,7 @@
 package org.opendaylight.controller.cluster.access.commands;
 
 import akka.actor.ActorRef;
+import java.io.Serial;
 import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier;
 
 /**
@@ -17,6 +18,7 @@ import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier
  * @author Robert Varga
  */
 final class TransactionPreCommitRequestProxyV1 extends AbstractTransactionRequestProxy<TransactionPreCommitRequest> {
+    @Serial
     private static final long serialVersionUID = 1L;
 
     // checkstyle flags the public modifier as redundant however it is explicitly needed for Java serialization to
index 716f37804cc0b88f7fd0efdba345fc8de381e086..a349d72e8b364a8c4c100f42dd7b0f2f1a034eb5 100644 (file)
@@ -7,6 +7,7 @@
  */
 package org.opendaylight.controller.cluster.access.commands;
 
+import java.io.Serial;
 import org.opendaylight.controller.cluster.access.ABIVersion;
 import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier;
 
@@ -16,6 +17,7 @@ import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier
  * @author Robert Varga
  */
 public final class TransactionPreCommitSuccess extends TransactionSuccess<TransactionPreCommitSuccess> {
+    @Serial
     private static final long serialVersionUID = 1L;
 
     public TransactionPreCommitSuccess(final TransactionIdentifier identifier, final long sequence) {
index 17b1b8da0d27cdaa473c73ad9608a8c2900bf8a3..4e71077f8352780e90be32a3f70d89a00ac767db 100644 (file)
@@ -7,6 +7,7 @@
  */
 package org.opendaylight.controller.cluster.access.commands;
 
+import java.io.Serial;
 import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier;
 
 /**
@@ -16,6 +17,7 @@ import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier
  * @author Robert Varga
  */
 final class TransactionPreCommitSuccessProxyV1 extends AbstractTransactionSuccessProxy<TransactionPreCommitSuccess> {
+    @Serial
     private static final long serialVersionUID = 1L;
 
     // checkstyle flags the public modifier as redundant however it is explicitly needed for Java serialization to
index ee56b4c81a66158efc06c7ddea430a29c7e1d7df..a6be95d9f96388b63abb7b84da3efff428f997fd 100644 (file)
@@ -8,6 +8,7 @@
 package org.opendaylight.controller.cluster.access.commands;
 
 import akka.actor.ActorRef;
+import java.io.Serial;
 import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier;
 
 /**
@@ -17,6 +18,7 @@ import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier
  * @author Robert Varga
  */
 final class TransactionPurgeRequestProxyV1 extends AbstractTransactionRequestProxy<TransactionPurgeRequest> {
+    @Serial
     private static final long serialVersionUID = 1L;
 
     // checkstyle flags the public modifier as redundant however it is explicitly needed for Java serialization to
index 54710143bcdd8633fa030c9f1b0040abffcb3fdd..1d3a74ab05a59953310b378733e803a6132109dc 100644 (file)
@@ -7,6 +7,7 @@
  */
 package org.opendaylight.controller.cluster.access.commands;
 
+import java.io.Serial;
 import org.opendaylight.controller.cluster.access.ABIVersion;
 import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier;
 
@@ -16,6 +17,7 @@ import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier
  * @author Robert Varga
  */
 public final class TransactionPurgeResponse extends TransactionSuccess<TransactionPurgeResponse> {
+    @Serial
     private static final long serialVersionUID = 1L;
 
     public TransactionPurgeResponse(final TransactionIdentifier identifier, final long sequence) {
index d15d7292b5c954f47162398084c3cbccd8ffcb7c..f51dc606fa96f1d11a2ecedbe3fbab9b0df62253 100644 (file)
@@ -7,6 +7,7 @@
  */
 package org.opendaylight.controller.cluster.access.commands;
 
+import java.io.Serial;
 import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier;
 
 /**
@@ -16,6 +17,7 @@ import org.opendaylight.controller.cluster.access.concepts.TransactionIdentifier
  * @author Robert Varga
  */
 final class TransactionPurgeResponseProxyV1 extends AbstractTransactionSuccessProxy<TransactionPurgeResponse> {
+    @Serial
     private static final long serialVersionUID = 1L;
 
     // checkstyle flags the public modifier as redundant however it is explicitly needed for Java serialization to
index 71a731addea1ad17ee275fc8056b793fb894c7ec..c4d100a560afaf388b99649945e7c9d4fd64add1 100644 (file)
@@ -11,9 +11,11 @@ import java.io.Externalizable;
 import java.io.IOException;
 import java.io.ObjectInput;
 import java.io.ObjectOutput;
+import java.io.Serial;
 import org.opendaylight.yangtools.concepts.WritableObjects;
 
 abstract class AbstractEnvelopeProxy<T extends Message<?, ?>> implements Externalizable {
+    @Serial
     private static final long serialVersionUID = 1L;
 
     private T message;
@@ -48,6 +50,7 @@ abstract class AbstractEnvelopeProxy<T extends Message<?, ?>> implements Externa
     @SuppressWarnings("checkstyle:hiddenField")
     abstract Envelope<T> createEnvelope(T wrappedNessage, long sessionId, long txSequence);
 
+    @Serial
     final Object readResolve() {
         return createEnvelope(message, sessionId, txSequence);
     }
index 0367527a5e67eb2ce796720786df1adeae733f1d..1cc091bdb28af6f2a5acdfa7c9b53a4b006f46ec 100644 (file)
@@ -14,6 +14,7 @@ import java.io.Externalizable;
 import java.io.IOException;
 import java.io.ObjectInput;
 import java.io.ObjectOutput;
+import java.io.Serial;
 import org.eclipse.jdt.annotation.NonNull;
 import org.opendaylight.yangtools.concepts.WritableIdentifier;
 import org.opendaylight.yangtools.concepts.WritableObjects;
@@ -27,7 +28,9 @@ import org.opendaylight.yangtools.concepts.WritableObjects;
  * @param <C> Message class
  */
 abstract class AbstractMessageProxy<T extends WritableIdentifier, C extends Message<T, C>> implements Externalizable {
+    @Serial
     private static final long serialVersionUID = 1L;
+
     private T target;
     private long sequence;
 
@@ -52,6 +55,7 @@ abstract class AbstractMessageProxy<T extends WritableIdentifier, C extends Mess
         sequence = WritableObjects.readLong(in);
     }
 
+    @Serial
     protected final Object readResolve() {
         return verifyNotNull(createMessage(target, sequence));
     }
index 1e873b4bdfc350b43d3a7b582523f98de9bfda3a..7c35a3fd53081c47dd8eb821398733e7f6c56496 100644 (file)
@@ -10,9 +10,11 @@ package org.opendaylight.controller.cluster.access.concepts;
 import java.io.IOException;
 import java.io.ObjectInput;
 import java.io.ObjectOutput;
+import java.io.Serial;
 import org.opendaylight.yangtools.concepts.WritableObjects;
 
 abstract class AbstractResponseEnvelopeProxy<T extends Response<?, ?>> extends AbstractEnvelopeProxy<T> {
+    @Serial
     private static final long serialVersionUID = 1L;
 
     private long executionTimeNanos;
index c9edfdb05fb5fd6ab7d3750ab4c4bcf1cf1a1828..818410dbb3921747b6f1dd691d307add02d9303e 100644 (file)
@@ -7,6 +7,7 @@
  */
 package org.opendaylight.controller.cluster.access.concepts;
 
+import java.io.Serial;
 import org.eclipse.jdt.annotation.NonNull;
 import org.opendaylight.yangtools.concepts.WritableIdentifier;
 
@@ -20,6 +21,7 @@ import org.opendaylight.yangtools.concepts.WritableIdentifier;
  */
 abstract class AbstractResponseProxy<T extends WritableIdentifier, C extends Response<T, C>>
         extends AbstractMessageProxy<T, C> {
+    @Serial
     private static final long serialVersionUID = 1L;
 
     protected AbstractResponseProxy() {
index 69e803ef42b7e2e31bbcfe81836d5ba42912ab7e..d62b59384562e85b9bc9bdbb8db5b1976aae15e6 100644 (file)
@@ -61,6 +61,7 @@ public final class ClientIdentifier implements WritableIdentifier {
             generation = WritableObjects.readLong(in);
         }
 
+        @Serial
         private Object readResolve() {
             return new ClientIdentifier(frontendId, generation);
         }
@@ -124,6 +125,7 @@ public final class ClientIdentifier implements WritableIdentifier {
             .toString();
     }
 
+    @Serial
     private Object writeReplace() {
         return new Proxy(frontendId, generation);
     }
index c6123d6fc35de45a4946dba21ac68f6a6b92dadc..44cf7171c7f902828b6d0c569a64e93431b5e537 100644 (file)
@@ -10,11 +10,13 @@ package org.opendaylight.controller.cluster.access.concepts;
 import static java.util.Objects.requireNonNull;
 
 import com.google.common.base.MoreObjects;
+import java.io.Serial;
 import java.io.Serializable;
 import org.eclipse.jdt.annotation.NonNull;
 import org.opendaylight.yangtools.concepts.Immutable;
 
 public abstract class Envelope<T extends Message<?, ?>> implements Immutable, Serializable {
+    @Serial
     private static final long serialVersionUID = 1L;
 
     private final @NonNull T message;
@@ -60,6 +62,7 @@ public abstract class Envelope<T extends Message<?, ?>> implements Immutable, Se
                 .add("txSequence", Long.toHexString(txSequence)).add("message", message).toString();
     }
 
+    @Serial
     final Object writeReplace() {
         return createProxy();
     }
index 1f641eb1819f945da32ee4a9b76a403e5e7bea3d..c2daa7420335f2440697c509ce5ddb23d960c17a 100644 (file)
@@ -7,7 +7,10 @@
  */
 package org.opendaylight.controller.cluster.access.concepts;
 
+import java.io.Serial;
+
 public final class FailureEnvelope extends ResponseEnvelope<RequestFailure<?, ?>> {
+    @Serial
     private static final long serialVersionUID = 1L;
 
     public FailureEnvelope(final RequestFailure<?, ?> message, final long sessionId, final long txSequence,
index adc50e1eaed216662135aa255b3849907adafc97..65c2f43b7e57b0d772b892bbe30d8c433c2e28d9 100644 (file)
@@ -7,7 +7,10 @@
  */
 package org.opendaylight.controller.cluster.access.concepts;
 
+import java.io.Serial;
+
 final class FailureEnvelopeProxy extends AbstractResponseEnvelopeProxy<RequestFailure<?, ?>> {
+    @Serial
     private static final long serialVersionUID = 1L;
 
     // checkstyle flags the public modifier as redundant however it is explicitly needed for Java serialization to be
index 00aa6cdd4b37781138fb1f8977c6c29cfca207ab..5afaa115b02ba7059e5aa793134057f38533ab2a 100644 (file)
@@ -55,6 +55,7 @@ public final class FrontendIdentifier implements WritableIdentifier {
             clientType = FrontendType.readFrom(in);
         }
 
+        @Serial
         private Object readResolve() {
             return new FrontendIdentifier(memberName, clientType);
         }
@@ -102,14 +103,8 @@ public final class FrontendIdentifier implements WritableIdentifier {
 
     @Override
     public boolean equals(final Object obj) {
-        if (this == obj) {
-            return true;
-        }
-        if (!(obj instanceof FrontendIdentifier other)) {
-            return false;
-        }
-
-        return memberName.equals(other.memberName) && clientType.equals(other.clientType);
+        return this == obj || obj instanceof FrontendIdentifier other && memberName.equals(other.memberName)
+            && clientType.equals(other.clientType);
     }
 
     public @NonNull String toPersistentId() {
@@ -121,6 +116,7 @@ public final class FrontendIdentifier implements WritableIdentifier {
         return toPersistentId();
     }
 
+    @Serial
     private Object writeReplace() {
         return new Proxy(memberName, clientType);
     }
index 755eda99bf6645435366a5db11d0746550f85fb6..b36cbbe4031c11b1ad8207e3f9a5a0339b78a1bb 100644 (file)
@@ -36,6 +36,7 @@ public final class FrontendType implements Comparable<FrontendType>, WritableIde
     private static final class Proxy implements Externalizable {
         @Serial
         private static final long serialVersionUID = 1L;
+
         private byte[] serialized;
 
         // checkstyle flags the public modifier as redundant however it is explicitly needed for Java serialization to
@@ -61,6 +62,7 @@ public final class FrontendType implements Comparable<FrontendType>, WritableIde
             in.readFully(serialized);
         }
 
+        @Serial
         private Object readResolve() {
             // TODO: consider caching instances here
             return new FrontendType(new String(serialized, StandardCharsets.UTF_8), serialized);
@@ -156,6 +158,7 @@ public final class FrontendType implements Comparable<FrontendType>, WritableIde
         return local;
     }
 
+    @Serial
     Object writeReplace() {
         return new Proxy(getSerialized());
     }
index 137bf5907f3ca985170f482d49ae48d86debbc56..e6050e76c5055f5517d1fef7674a27b269147121 100644 (file)
@@ -16,6 +16,7 @@ import java.io.Externalizable;
 import java.io.IOException;
 import java.io.ObjectInput;
 import java.io.ObjectOutput;
+import java.io.Serial;
 import org.eclipse.jdt.annotation.NonNull;
 import org.opendaylight.yangtools.concepts.WritableIdentifier;
 import org.opendaylight.yangtools.concepts.WritableObjects;
@@ -25,8 +26,6 @@ import org.opendaylight.yangtools.concepts.WritableObjects;
  * - a {@link ClientIdentifier}, which uniquely identifies a single instantiation of a particular frontend
  * - an unsigned long, which uniquely identifies the history on the backend
  * - an unsigned long cookie, assigned by the client and meaningless on the backend, which just reflects it back
- *
- * @author Robert Varga
  */
 public final class LocalHistoryIdentifier implements WritableIdentifier {
     /*
@@ -39,7 +38,9 @@ public final class LocalHistoryIdentifier implements WritableIdentifier {
      *                      compatibility.
      */
     private static final class Proxy implements Externalizable {
+        @Serial
         private static final long serialVersionUID = 1L;
+
         private ClientIdentifier clientId;
         private long historyId;
         private long cookie;
@@ -72,12 +73,15 @@ public final class LocalHistoryIdentifier implements WritableIdentifier {
             cookie = WritableObjects.readSecondLong(in, header);
         }
 
+        @Serial
         private Object readResolve() {
             return new LocalHistoryIdentifier(clientId, historyId, cookie);
         }
     }
 
+    @Serial
     private static final long serialVersionUID = 1L;
+
     private final @NonNull ClientIdentifier clientId;
     private final long historyId;
     private final long cookie;
@@ -131,11 +135,10 @@ public final class LocalHistoryIdentifier implements WritableIdentifier {
         if (this == obj) {
             return true;
         }
-        if (!(obj instanceof LocalHistoryIdentifier)) {
+        if (!(obj instanceof LocalHistoryIdentifier other)) {
             return false;
         }
 
-        final LocalHistoryIdentifier other = (LocalHistoryIdentifier) obj;
         return historyId == other.historyId && cookie == other.cookie && clientId.equals(other.clientId);
     }
 
@@ -146,6 +149,7 @@ public final class LocalHistoryIdentifier implements WritableIdentifier {
                 .add("cookie", Long.toUnsignedString(cookie, 16)).toString();
     }
 
+    @Serial
     private Object writeReplace() {
         return new Proxy(clientId, historyId, cookie);
     }
index 2404924c640b68970dda49578527b33de974035a..f26df831836cc7852cf31f9fd8a826199ffa70b1 100644 (file)
@@ -58,6 +58,7 @@ public final class MemberName implements Comparable<MemberName>, WritableIdentif
             in.readFully(serialized);
         }
 
+        @Serial
         private Object readResolve() {
             // TODO: consider caching instances here
             return new MemberName(new String(serialized, StandardCharsets.UTF_8), serialized);
@@ -140,6 +141,7 @@ public final class MemberName implements Comparable<MemberName>, WritableIdentif
         return local;
     }
 
+    @Serial
     Object writeReplace() {
         return new Proxy(getSerialized());
     }
index 9b1c4c25b3d046847adad91c113c8dadaaeedf84..6ee547fc93e75a8acf80a4b0e2f5c4f8ace4be55 100644 (file)
@@ -110,8 +110,6 @@ public abstract class Message<T extends WritableIdentifier, C extends Message<T,
 
         return switch (toVersion) {
             case BORON, NEON_SR2, SODIUM_SR1, MAGNESIUM -> verifyNotNull(cloneAsVersion(toVersion));
-            case TEST_PAST_VERSION, TEST_FUTURE_VERSION ->
-                throw new IllegalArgumentException("Unhandled ABI version " + toVersion);
             default -> throw new IllegalArgumentException("Unhandled ABI version " + toVersion);
         };
     }
@@ -154,6 +152,7 @@ public abstract class Message<T extends WritableIdentifier, C extends Message<T,
      */
     abstract @NonNull AbstractMessageProxy<T, C> externalizableProxy(@NonNull ABIVersion reqVersion);
 
+    @Serial
     protected final Object writeReplace() {
         return externalizableProxy(version);
     }
index 46d5d1f99620b83304b49e7d160a6648768dce65..4dac0b0330d595fdbf7ae5daac0c7f7008257c20 100644 (file)
@@ -8,8 +8,10 @@
 package org.opendaylight.controller.cluster.access.concepts;
 
 import akka.actor.ActorRef;
+import java.io.Serial;
 
 public final class RequestEnvelope extends Envelope<Request<?, ?>> {
+    @Serial
     private static final long serialVersionUID = 1L;
 
     public RequestEnvelope(final Request<?, ?> message, final long sessionId, final long txSequence) {
index 66e7eaa0a2fa28268deabc33f0ccda5b69c24655..21b99807af77c36f59c3390c68bf240f5a02b599 100644 (file)
@@ -7,7 +7,10 @@
  */
 package org.opendaylight.controller.cluster.access.concepts;
 
+import java.io.Serial;
+
 final class RequestEnvelopeProxy extends AbstractEnvelopeProxy<Request<?, ?>> {
+    @Serial
     private static final long serialVersionUID = 1L;
 
     // checkstyle flags the public modifier as redundant however it is explicitly needed for Java serialization to
index 7936baa1696059b184b8ff55f548ac5fd6e9229b..90fe7beb11e72a92111f4eeea323aaa8c48b9516 100644 (file)
@@ -8,8 +8,10 @@
 package org.opendaylight.controller.cluster.access.concepts;
 
 import com.google.common.base.Preconditions;
+import java.io.Serial;
 
 public abstract class ResponseEnvelope<T extends Response<?, ?>> extends Envelope<T> {
+    @Serial
     private static final long serialVersionUID = 1L;
 
     private final long executionTimeNanos;
index 3c23a23763cd071abb2e967f7c58b6e9cd633ca1..bf35c4931f00d402acbd6071b66ce1379618c5ad 100644 (file)
@@ -7,7 +7,10 @@
  */
 package org.opendaylight.controller.cluster.access.concepts;
 
+import java.io.Serial;
+
 public final class SuccessEnvelope extends ResponseEnvelope<RequestSuccess<?, ?>> {
+    @Serial
     private static final long serialVersionUID = 1L;
 
     public SuccessEnvelope(final RequestSuccess<?, ?> message, final long sessionId, final long txSequence,
index 3ac388b9dba2c5143558c7cde17657bb3d09f949..76adbb193f02e082072235808eaaa7d9a475f956 100644 (file)
@@ -7,7 +7,10 @@
  */
 package org.opendaylight.controller.cluster.access.concepts;
 
+import java.io.Serial;
+
 final class SuccessEnvelopeProxy extends AbstractResponseEnvelopeProxy<RequestSuccess<?, ?>> {
+    @Serial
     private static final long serialVersionUID = 1L;
 
     // checkstyle flags the public modifier as redundant however it is explicitly needed for Java serialization to
index 354cf8c9bf216454623202415b702bbf8d512efa..29edfe8fda05861a6af1a44b274924b273a89210 100644 (file)
@@ -55,6 +55,7 @@ public final class TransactionIdentifier implements WritableIdentifier {
             transactionId = WritableObjects.readLong(in);
         }
 
+        @Serial
         private Object readResolve() {
             return new TransactionIdentifier(historyId, transactionId);
         }
@@ -119,6 +120,7 @@ public final class TransactionIdentifier implements WritableIdentifier {
         return toShortString();
     }
 
+    @Serial
     private Object writeReplace() {
         return new Proxy(historyId, transactionId);
     }