*/
package org.opendaylight.controller.cluster.access.concepts;
+import static java.util.Objects.requireNonNull;
+
import com.google.common.annotations.Beta;
-import com.google.common.base.MoreObjects;
-import com.google.common.base.Preconditions;
+import java.io.DataInput;
+import java.io.DataOutput;
import java.io.Externalizable;
import java.io.IOException;
import java.io.ObjectInput;
import java.io.ObjectOutput;
-import javax.annotation.Nonnull;
-import org.opendaylight.yangtools.concepts.Identifier;
+import org.eclipse.jdt.annotation.NonNull;
+import org.opendaylight.yangtools.concepts.WritableIdentifier;
+import org.opendaylight.yangtools.concepts.WritableObjects;
/**
* Globally-unique identifier of a transaction.
*
- * @param <T> Frontend type
- *
* @author Robert Varga
*/
@Beta
-public final class TransactionIdentifier<T extends FrontendType> implements Identifier {
- private static final class Proxy<T extends FrontendType> implements Externalizable {
+public final class TransactionIdentifier implements WritableIdentifier {
+ private static final class Proxy implements Externalizable {
private static final long serialVersionUID = 1L;
- private LocalHistoryIdentifier<T> historyId;
+ private LocalHistoryIdentifier historyId;
private long transactionId;
+ // checkstyle flags the public modifier as redundant however it is explicitly needed for Java serialization to
+ // be able to create instances via reflection.
+ @SuppressWarnings("checkstyle:RedundantModifier")
public Proxy() {
// For Externalizable
}
- Proxy(final LocalHistoryIdentifier<T> historyId, final long transactionId) {
- this.historyId = Preconditions.checkNotNull(historyId);
+ Proxy(final LocalHistoryIdentifier historyId, final long transactionId) {
+ this.historyId = requireNonNull(historyId);
this.transactionId = transactionId;
}
@Override
public void writeExternal(final ObjectOutput out) throws IOException {
- out.writeObject(historyId);
- out.writeLong(transactionId);
+ historyId.writeTo(out);
+ WritableObjects.writeLong(out, transactionId);
}
- @SuppressWarnings("unchecked")
@Override
- public void readExternal(final ObjectInput in) throws IOException, ClassNotFoundException {
- historyId = (LocalHistoryIdentifier<T>) in.readObject();
- transactionId = in.readLong();
+ public void readExternal(final ObjectInput in) throws IOException {
+ historyId = LocalHistoryIdentifier.readFrom(in);
+ transactionId = WritableObjects.readLong(in);
}
private Object readResolve() {
- return new TransactionIdentifier<>(historyId, transactionId);
+ return new TransactionIdentifier(historyId, transactionId);
}
}
private static final long serialVersionUID = 1L;
- private final LocalHistoryIdentifier<T> historyId;
+ private final @NonNull LocalHistoryIdentifier historyId;
private final long transactionId;
+ private String shortString;
- public TransactionIdentifier(final @Nonnull LocalHistoryIdentifier<T> historyId, final long transactionId) {
- this.historyId = Preconditions.checkNotNull(historyId);
+ public TransactionIdentifier(final @NonNull LocalHistoryIdentifier historyId, final long transactionId) {
+ this.historyId = requireNonNull(historyId);
this.transactionId = transactionId;
}
- public LocalHistoryIdentifier<T> getHistoryId() {
+ public static @NonNull TransactionIdentifier readFrom(final DataInput in) throws IOException {
+ final LocalHistoryIdentifier historyId = LocalHistoryIdentifier.readFrom(in);
+ return new TransactionIdentifier(historyId, WritableObjects.readLong(in));
+ }
+
+ @Override
+ public void writeTo(final DataOutput out) throws IOException {
+ historyId.writeTo(out);
+ WritableObjects.writeLong(out, transactionId);
+ }
+
+ public @NonNull LocalHistoryIdentifier getHistoryId() {
return historyId;
}
}
@Override
- public boolean equals(final Object o) {
- if (this == o) {
+ public boolean equals(final Object obj) {
+ if (this == obj) {
return true;
}
- if (!(o instanceof TransactionIdentifier)) {
+ if (!(obj instanceof TransactionIdentifier)) {
return false;
}
- final TransactionIdentifier<?> other = (TransactionIdentifier<?>) o;
+ final TransactionIdentifier other = (TransactionIdentifier) obj;
return transactionId == other.transactionId && historyId.equals(other.historyId);
}
+ public String toShortString() {
+ if (shortString == null) {
+ String histStr = historyId.getHistoryId() == 0 ? "" : "-chn-" + historyId.getHistoryId();
+ shortString = historyId.getClientId().getFrontendId().getMemberName().getName() + "-"
+ + historyId.getClientId().getFrontendId().getClientType().getName() + "-fe-"
+ + historyId.getClientId().getGeneration() + histStr + "-txn-" + transactionId
+ + "-" + historyId.getCookie();
+ }
+
+ return shortString;
+ }
+
@Override
public String toString() {
- return MoreObjects.toStringHelper(TransactionIdentifier.class).add("history", historyId)
- .add("transaction", transactionId).toString();
+ return toShortString();
}
private Object writeReplace() {
- return new Proxy<>(historyId, transactionId);
+ return new Proxy(historyId, transactionId);
}
}