You are viewing a plain text version of this content. The canonical link for it is here.
Posted to gitbox@activemq.apache.org by GitBox <gi...@apache.org> on 2021/08/06 10:48:50 UTC

[GitHub] [activemq-artemis] franz1981 opened a new pull request #3687: ARTEMIS-3340 Atomic server restart and clean dirty activation: fix testFailbackTimeout

franz1981 opened a new pull request #3687:
URL: https://github.com/apache/activemq-artemis/pull/3687


   It's going to fix intermittent failures of `testFailbackTimeout`:
   
   - Server restart is now atomic with other server operations (start/stop): it won't make a racy stop to be lost. This is mostly affecting tests that try stop a server while being restarted
   - Existing replication mechanism is very asynchronous by design and a live stop while backup data is not fully in sync can cause the backup to restart believing to have "good" in-sync data (because the activation sequence and NodeID are fine); this can affect both test that interrupt a syncing process mid-air AND real world. 
   
   The latter requires more attention in a separate JIRA, because if backup crash after written the right NodeID and activation sequence (and live was stopped/crashed, before incrementing the coordinated sequence) it can still be restarted and, if the data is good, believe to be a candidate to be live (maybe is correct, but need some deeper investigation!).
   The current fix just reset the local activation sequence to let this broker restart without activating by accident.
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: gitbox-unsubscribe@activemq.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [activemq-artemis] clebertsuconic merged pull request #3687: ARTEMIS-3340 Atomic server restart and clean dirty activation: fix testFailbackTimeout

Posted by GitBox <gi...@apache.org>.
clebertsuconic merged pull request #3687:
URL: https://github.com/apache/activemq-artemis/pull/3687


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: gitbox-unsubscribe@activemq.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org