[ 
https://issues.apache.org/jira/browse/IGNITE-26397?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Vladislav Pyatkov updated IGNITE-26397:
---------------------------------------
    Description: 
h3. Motivation
The initial value of property _shouldBeSavedForRecovery_ always the same as 
with _NetworkMessage#needAck_. It also makes a disadvantage when we add a 
message without acknowledgment because we have to change a constructor as well 
(use one with 3 params).

h3. Definition of done
Leave only a constructor with 2 params.
Property _shouldBeSavedForRecovery_ should be initialized by needAck.

  was:
h3. Motivation
The initial value of property `shouldBeSavedForRecovery` always the same as 
with `NetworkMessage#needAck`. It also makes a disadvantage when we add a 
message without acknowledgment because we have to change a constructor as well 
(use one with 3 params).

h3. Definition of done
Leave only a constructor with 2 params.
Property `shouldBeSavedForRecovery` should be initialized by needAck.


> OutNetworkObject constructor with shouldBeSavedForRecovery is redundant
> -----------------------------------------------------------------------
>
>                 Key: IGNITE-26397
>                 URL: https://issues.apache.org/jira/browse/IGNITE-26397
>             Project: Ignite
>          Issue Type: Improvement
>            Reporter: Vladislav Pyatkov
>            Assignee: Vladislav Pyatkov
>            Priority: Major
>              Labels: ignite-3
>
> h3. Motivation
> The initial value of property _shouldBeSavedForRecovery_ always the same as 
> with _NetworkMessage#needAck_. It also makes a disadvantage when we add a 
> message without acknowledgment because we have to change a constructor as 
> well (use one with 3 params).
> h3. Definition of done
> Leave only a constructor with 2 params.
> Property _shouldBeSavedForRecovery_ should be initialized by needAck.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to