[ARTEMIS-5010] AckManager records from mirror are not being replicated

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.37.0
• Fix Version/s: 2.38.0
• Assignee: Unassigned
• Reporter: Clebert Suconic
• Created: 23/Aug/24 01:38
• Updated: 16/Oct/24 16:02
• Resolved: 28/Aug/24 18:08

Description

The overall problems is that JournalHashMap is using a local journal most of the time and never the replicated journal.

In this line:

https://github.com/apache/activemq-artemis/blob/38693370c962f9a4d7fbc9dbeb7bffba18f9ad41/artemis-protocols/artemis-amqp-protocol/src/main/java/org/apache/activemq/artemis/protocol/amqp/connect/mirror/AckManager.java#L82

We send the current journal.. and if a switch happens inside the JournalStorageManager the switch is never captured, and we will never replicate pending acks.

When a failover happens the replica will not have the retries and the system will not reattempt them.

Issue Links

• incorporates: ARTEMIS-4971 AckManager "giving up" processing of Acks in warning log level
• links to: GitHub Pull Request #5173
• links to: GitHub Pull Request #5186
• links to: GitHub Pull Request #5213
• links to: GitHub Pull Request #5224