[AMQ-6517] Make redelivery policy preDispatch check optional

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 5.14.0
• Fix Version/s: 5.15.0
• Component/s: JMS client
• Assignee: Gary Tully
• Reporter: Gary Tully
• Created: 24/Nov/16 16:38
• Updated: 22/Feb/17 14:35
• Resolved: 24/Nov/16 17:03

Description

AMQ-5146 introduced a pre dispatch check on the redelivery policy to ensure that a consumer the repeatidly crashed with a redelivery limit would eventually fail the message to the dlq.
There are cases where this is not ideal. For example, if the redelivery policy denies redelivery, maxRedeliveries=0 and a message has been inflight on an aborted connection. Broker side the redelivery count will be incremented, because the broker knows it may have been delivered (on a connection abort, the consumer goes not get to supply accurate delivery status information). With the preemptive check, this message it won't get delivered and will instead go directly to the DLQ.

making the preDeliveryCheck optional allows maxRedeliveries=0 to work in this case.

Issue Links

• is related to: AMQ-5146 RedeliveryPolicy exceeded for crashing consumer