[ARTEMIS-4558] AMQP Mirror ACKS should be idempotent

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 2.32.0
• Assignee: Clebert Suconic
• Reporter: Clebert Suconic
• Created: 08/Jan/24 23:12
• Updated: 24/Jan/24 18:35
• Resolved: 24/Jan/24 18:35

Description

When I first developed Mirroring, I assumed sending the mirrored ACK on a aferACK and disconnected from any other transactions would be enough, with the caveat you could get a duplicate delivery on the target mirror in case of failures.

I got some complains that this is not safe enough from some users, and I'm making this now idempotent.

I took an overal mirroring hardening approach and I'm improving test coverage for this improvement.

Issue Links

• links to: GitHub Pull Request #4734