[ 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)