[
https://issues.apache.org/jira/browse/QPIDJMS-274?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15935411#comment-15935411
]
Timothy Bish commented on QPIDJMS-274:
--------------------------------------
On the surface of it I think this is okay to add, however I'd recommend some
more work on the patch. The terminal outcome applied by the policy doesn't
appear as though it can be set via the URI which makes it less user friendly.
I'd rename the ackType value in the policy to terminalOutcome to better match
what it's purpose is as well. There's no tests of the new functionality either
so we don't even know if it works so adding test coverage would be good.
I'd expect to be able to do something like
{code}?jms.redeliveryPolicy.terminalOutcome="REJECTED"{code} on the URI and I'd
hope there was some validation on the value to prevent users from setting the
values from ACK_TYPE that don't make sense here.
> Be able to REJECT messages in JmsRedeliveryPolicy
> -------------------------------------------------
>
> Key: QPIDJMS-274
> URL: https://issues.apache.org/jira/browse/QPIDJMS-274
> Project: Qpid JMS
> Issue Type: Improvement
> Components: qpid-jms-client
> Affects Versions: 0.21.0
> Reporter: Fredrik Jönsson
>
> This is a placeholder for discussion right now, I believe I could implement
> it myself, but would like suggestions as to how to do so in an acceptable
> manner for inclusion in the project in that case if there are opinions about
> it.
> I would like to optionally (by configuration) *reject* messages with
> JmsRedeliveryPolicy/DefaultJmsRedeliveryPolicy, rather than respond with
> modify_undeliverable.
> The particular use case I have is a client reading from Azure Service Bus. I
> want it to be able to reject messages that it knows to be undeliverable to
> explicitly dead-letter them immediately.
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
