(ignite) branch master updated: IGNITE-28574 Fix error deserialization in ServiceSingleNodeDeploymentResult (#13048)

[av]

This is an automated email from the ASF dual-hosted git repository.

anton-vinogradov pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/ignite.git


The following commit(s) were added to refs/heads/master by this push:
     new d0a1ae09a11 IGNITE-28574 Fix error deserialization in 
ServiceSingleNodeDeploymentResult (#13048)
d0a1ae09a11 is described below

commit d0a1ae09a114ce8131e5d5a44f6b9915bbb829dd
Author: Dmitry Werner <grimekil...@gmail.com>
AuthorDate: Sat Apr 18 21:09:09 2026 +0500

    IGNITE-28574 Fix error deserialization in ServiceSingleNodeDeploymentResult 
(#13048)
---
 .../service/ServiceSingleNodeDeploymentResult.java | 23 +++++++++++-----------
 1 file changed, 11 insertions(+), 12 deletions(-)

diff --git 
a/modules/core/src/main/java/org/apache/ignite/internal/processors/service/ServiceSingleNodeDeploymentResult.java
 
b/modules/core/src/main/java/org/apache/ignite/internal/processors/service/ServiceSingleNodeDeploymentResult.java
index 8fc1530529c..ad92ea54f8c 100644
--- 
a/modules/core/src/main/java/org/apache/ignite/internal/processors/service/ServiceSingleNodeDeploymentResult.java
+++ 
b/modules/core/src/main/java/org/apache/ignite/internal/processors/service/ServiceSingleNodeDeploymentResult.java
@@ -99,9 +99,7 @@ public class ServiceSingleNodeDeploymentResult implements 
MarshallableMessage, S
 
         for (Throwable th : errors) {
             try {
-                byte[] arr = U.marshal(marsh, th);
-
-                errorsBytes.add(arr);
+                errorsBytes.add(U.marshal(marsh, th));
             }
             catch (IgniteCheckedException e) {
                 log.error("Failed to marshal deployment error, err=" + th, e);
@@ -123,18 +121,19 @@ public class ServiceSingleNodeDeploymentResult implements 
MarshallableMessage, S
 
     /** {@inheritDoc} */
     @Override public void finishUnmarshal(Marshaller marsh, ClassLoader 
clsLdr) throws IgniteCheckedException {
-        try {
-            if (errorsBytes != null && errors == null) {
-                errors = new ArrayList<>();
+        if (errorsBytes != null && errors == null) {
+            errors = new ArrayList<>();
 
-                for (byte[] arr : errorsBytes)
+            for (byte[] arr : errorsBytes) {
+                try {
                     errors.add(U.unmarshal(marsh, arr, clsLdr));
-            }
-        }
-        catch (IgniteCheckedException e) {
-            U.error(null, "Failed to unmarshal deployment result message", e);
+                }
+                catch (IgniteCheckedException e) {
+                    U.error(null, "Failed to unmarshal deployment error.", e);
 
-            errors.add(new IgniteCheckedException("Failed to unmarshal 
deployment error, see server logs for details."));
+                    errors.add(new IgniteCheckedException("Failed to unmarshal 
deployment error, see server logs for details."));
+                }
+            }
         }
     }