[
https://issues.apache.org/jira/browse/ARTEMIS-2494?focusedWorklogId=318228&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-318228
]
ASF GitHub Bot logged work on ARTEMIS-2494:
-------------------------------------------
Author: ASF GitHub Bot
Created on: 25/Sep/19 10:43
Start Date: 25/Sep/19 10:43
Worklog Time Spent: 10m
Work Description: k-wall commented on issue #2847: ARTEMIS-2494: [AMQP]
Allow Modified disposition to be used signal address full to a sending peer
URL: https://github.com/apache/activemq-artemis/pull/2847#issuecomment-534963469
My initial patch missed the code path where the address full policy is BLOCK
and the paging store signals itself full. Previously reject would be sent
unconditionally. Now Modified is sent if enabled/supported on the link as per
the other case.
There is a small behavioural change in this patch (a25a289). Previously on
the BLOCK code path the Reject for an address-full would be sent with a
`resource-limit-exceeded `whereas the other code path would send a Reject for
an address-full with a condition `failed`. This patch uses
`resource-limit-exceeded` consistently.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
Issue Time Tracking
-------------------
Worklog Id: (was: 318228)
Time Spent: 3h 10m (was: 3h)
> Artemis responds with disposition Rejected if queue is full
> -----------------------------------------------------------
>
> Key: ARTEMIS-2494
> URL: https://issues.apache.org/jira/browse/ARTEMIS-2494
> Project: ActiveMQ Artemis
> Issue Type: Bug
> Components: AMQP
> Affects Versions: 2.10.0
> Reporter: Ulf Lilleengen
> Priority: Major
> Time Spent: 3h 10m
> Remaining Estimate: 0h
>
> When a queue is full, Artemis will respond with disposition=Rejected. In
> certain situations, it is desirable to receive a Modified in this case, so
> that a client may re-queue the message for re-delivery (this is what Artemis
> does if consumers respond with released or modified for instance).
>
> The current behavior is determined by
> ProtonServerReceiverContext#actualDelivery . It would be great if this method
> could respond with Modified instead in the event of the exception being an
> ActiveMQAddressFullException.
>
> This behavior should probably be exposed as a configuration option to avoid
> breaking existing behavior.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
