NSAmelchev commented on code in PR #12895:
URL: https://github.com/apache/ignite/pull/12895#discussion_r3001645067
##########
modules/core/src/main/java/org/apache/ignite/internal/processors/service/ServiceClusterDeploymentResult.java:
##########
@@ -17,30 +17,36 @@
package org.apache.ignite.internal.processors.service;
-import java.io.Serializable;
import java.util.Collections;
import java.util.Map;
import java.util.UUID;
+import org.apache.ignite.internal.Order;
import org.apache.ignite.internal.util.tostring.GridToStringInclude;
import org.apache.ignite.internal.util.typedef.internal.S;
import org.apache.ignite.lang.IgniteUuid;
+import org.apache.ignite.plugin.extensions.communication.Message;
import org.jetbrains.annotations.NotNull;
/**
* Service cluster deployment result.
* <p/>
* Contains coint of deployed service and deployment errors across the cluster
mapped to nodes ids.
*/
-public class ServiceClusterDeploymentResult implements Serializable {
- /** */
- private static final long serialVersionUID = 0L;
-
+public class ServiceClusterDeploymentResult implements Message {
/** Service id. */
- private final IgniteUuid srvcId;
+ @Order(0)
+ IgniteUuid srvcId;
/** Per node deployments results. */
+ @Order(1)
@GridToStringInclude
- private final Map<UUID, ServiceSingleNodeDeploymentResult> results;
+ Map<UUID, ServiceSingleNodeDeploymentResult> results;
+
+ /**
+ * Empty constructor for marshalling purposes.
+ */
Review Comment:
```suggestion
/** Default constructor for {@link MessageFactory}. */
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
