[
https://issues.apache.org/jira/browse/ARTEMIS-5107?focusedWorklogId=953336&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-953336
]
ASF GitHub Bot logged work on ARTEMIS-5107:
-------------------------------------------
Author: ASF GitHub Bot
Created on: 21/Jan/25 13:36
Start Date: 21/Jan/25 13:36
Worklog Time Spent: 10m
Work Description: gemmellr commented on code in PR #5451:
URL: https://github.com/apache/activemq-artemis/pull/5451#discussion_r1923743956
##########
artemis-server/src/main/java/org/apache/activemq/artemis/core/protocol/core/impl/wireformat/ReplicationStartSyncMessage.java:
##########
@@ -76,7 +76,7 @@ public static SyncDataType getDataType(byte code) {
public ReplicationStartSyncMessage(boolean beforeTwoEighteen) {
super(REPLICATION_START_FINISH_SYNC);
- this.beforeTwoEighteen = synchronizationIsFinished;
+ this.beforeTwoEighteen = beforeTwoEighteen;
Review Comment:
If _this.beforeTwoEighteen_ was always being set false before, due to the
_synchronizationIsFinished_ field being false at this point, what are the
implications of changing this now such that it might now become true?
This seems to point to a test either not existing or not actually being
effective in checking this, or perhaps just not actually needing this/related
code if it never actually did anything. It doesnt seem like this change should
be happening without some other matching change.
Issue Time Tracking
-------------------
Worklog Id: (was: 953336)
Time Spent: 20m (was: 10m)
> Using uninitialized value in ReplicationStartSyncMessage
> --------------------------------------------------------
>
> Key: ARTEMIS-5107
> URL: https://issues.apache.org/jira/browse/ARTEMIS-5107
> Project: ActiveMQ Artemis
> Issue Type: Bug
> Reporter: Pavel Nekrasov
> Assignee: Justin Bertram
> Priority: Major
> Labels: pull-request-available
> Time Spent: 20m
> Remaining Estimate: 0h
>
> In lines
> [77-80|https://github.com/apache/activemq-artemis/blob/8e4bc33dc4d7b4d2031c1bda996c7ce002a3d0e8/artemis-server/src/main/java/org/apache/activemq/artemis/core/protocol/core/impl/wireformat/ReplicationStartSyncMessage.java#L77-L80],
> an uninitialized value {{synchronizationIsFinished}} is assigned, while the
> {{beforeTwoEighteen}} parameter passed to the constructor is not used.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
For further information, visit: https://activemq.apache.org/contact
