[ 
https://issues.apache.org/jira/browse/ARTEMIS-5107?focusedWorklogId=953705&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-953705
 ]

ASF GitHub Bot logged work on ARTEMIS-5107:
-------------------------------------------

                Author: ASF GitHub Bot
            Created on: 22/Jan/25 17:50
            Start Date: 22/Jan/25 17:50
    Worklog Time Spent: 10m 
      Work Description: jbertram commented on code in PR #5451:
URL: https://github.com/apache/activemq-artemis/pull/5451#discussion_r1925733494


##########
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:
   As I understand it, the problem wasn't so much that the newer server was 
sending _additional_ data so much as it was sending _different_ data that the 
older server was choking on.





Issue Time Tracking
-------------------

    Worklog Id:     (was: 953705)
    Time Spent: 1h  (was: 50m)

> 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: 1h
>  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


Reply via email to