Skip to content

Commit bef13bb

Browse files
committed
IGNITE-27333 Use MessageSerializer for TcpDiscoveryClientAckResponse
1 parent 198d433 commit bef13bb

File tree

2 files changed

+25
-2
lines changed

2 files changed

+25
-2
lines changed

modules/core/src/main/java/org/apache/ignite/internal/managers/discovery/DiscoveryMessageFactory.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@
1818
package org.apache.ignite.internal.managers.discovery;
1919

2020
import org.apache.ignite.internal.codegen.TcpDiscoveryCheckFailedMessageSerializer;
21+
import org.apache.ignite.internal.codegen.TcpDiscoveryClientAckResponseSerializer;
2122
import org.apache.ignite.internal.codegen.TcpDiscoveryClientPingRequestSerializer;
2223
import org.apache.ignite.internal.codegen.TcpDiscoveryClientPingResponseSerializer;
2324
import org.apache.ignite.internal.codegen.TcpDiscoveryLoopbackProblemMessageSerializer;
@@ -26,6 +27,7 @@
2627
import org.apache.ignite.plugin.extensions.communication.MessageFactory;
2728
import org.apache.ignite.plugin.extensions.communication.MessageFactoryProvider;
2829
import org.apache.ignite.spi.discovery.tcp.messages.TcpDiscoveryCheckFailedMessage;
30+
import org.apache.ignite.spi.discovery.tcp.messages.TcpDiscoveryClientAckResponse;
2931
import org.apache.ignite.spi.discovery.tcp.messages.TcpDiscoveryClientPingRequest;
3032
import org.apache.ignite.spi.discovery.tcp.messages.TcpDiscoveryClientPingResponse;
3133
import org.apache.ignite.spi.discovery.tcp.messages.TcpDiscoveryLoopbackProblemMessage;
@@ -42,5 +44,6 @@ public class DiscoveryMessageFactory implements MessageFactoryProvider {
4244
factory.register((short)3, TcpDiscoveryClientPingRequest::new, new TcpDiscoveryClientPingRequestSerializer());
4345
factory.register((short)4, TcpDiscoveryClientPingResponse::new, new TcpDiscoveryClientPingResponseSerializer());
4446
factory.register((short)5, TcpDiscoveryLoopbackProblemMessage::new, new TcpDiscoveryLoopbackProblemMessageSerializer());
47+
factory.register((short)6, TcpDiscoveryClientAckResponse::new, new TcpDiscoveryClientAckResponseSerializer());
4548
}
4649
}

modules/core/src/main/java/org/apache/ignite/spi/discovery/tcp/messages/TcpDiscoveryClientAckResponse.java

Lines changed: 22 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,18 +18,26 @@
1818
package org.apache.ignite.spi.discovery.tcp.messages;
1919

2020
import java.util.UUID;
21+
import org.apache.ignite.internal.Order;
2122
import org.apache.ignite.internal.util.typedef.internal.S;
2223
import org.apache.ignite.lang.IgniteUuid;
24+
import org.apache.ignite.plugin.extensions.communication.Message;
2325

2426
/**
2527
*
2628
*/
27-
public class TcpDiscoveryClientAckResponse extends TcpDiscoveryAbstractMessage {
29+
public class TcpDiscoveryClientAckResponse extends TcpDiscoveryAbstractMessage implements Message {
2830
/** */
2931
private static final long serialVersionUID = 0L;
3032

3133
/** */
32-
private final IgniteUuid msgId;
34+
@Order(value = 5, method = "messageId")
35+
private IgniteUuid msgId;
36+
37+
/** */
38+
public TcpDiscoveryClientAckResponse() {
39+
// No-op.
40+
}
3341

3442
/**
3543
* @param creatorNodeId Creator node ID.
@@ -48,6 +56,13 @@ public IgniteUuid messageId() {
4856
return msgId;
4957
}
5058

59+
/**
60+
* @param msgId Acknowledged message ID.
61+
*/
62+
public void messageId(IgniteUuid msgId) {
63+
this.msgId = msgId;
64+
}
65+
5166
/** {@inheritDoc} */
5267
@Override public boolean traceLogLevel() {
5368
return true;
@@ -62,4 +77,9 @@ public IgniteUuid messageId() {
6277
@Override public String toString() {
6378
return S.toString(TcpDiscoveryClientAckResponse.class, this, "super", super.toString());
6479
}
80+
81+
/** {@inheritDoc} */
82+
@Override public short directType() {
83+
return 6;
84+
}
6585
}

0 commit comments

Comments
 (0)