[
https://issues.apache.org/jira/browse/ARTEMIS-2649?focusedWorklogId=437440&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-437440
]
ASF GitHub Bot logged work on ARTEMIS-2649:
-------------------------------------------
Author: ASF GitHub Bot
Created on: 27/May/20 03:50
Start Date: 27/May/20 03:50
Worklog Time Spent: 10m
Work Description: clebertsuconic commented on a change in pull request
#3017:
URL: https://github.com/apache/activemq-artemis/pull/3017#discussion_r430822358
##########
File path:
artemis-core-client/src/main/java/org/apache/activemq/artemis/api/core/Message.java
##########
@@ -463,30 +463,40 @@ default void reencode() {
}
default void referenceOriginalMessage(final Message original, String
originalQueue) {
- String queueOnMessage =
original.getAnnotationString(Message.HDR_ORIGINAL_QUEUE);
+ String originalQueueOnMessage =
original.getAnnotationString(Message.HDR_ORIGINAL_QUEUE);
- if (queueOnMessage != null) {
- setAnnotation(Message.HDR_ORIGINAL_QUEUE, queueOnMessage);
- } else if (originalQueue != null) {
- setAnnotation(Message.HDR_ORIGINAL_QUEUE, originalQueue);
+ if (originalQueueOnMessage != null) {
+ setBreadcrumb(Message.HDR_ORIGINAL_QUEUE, originalQueueOnMessage);
}
+ setAnnotation(Message.HDR_ORIGINAL_QUEUE, originalQueue);
+
Object originalID = original.getAnnotation(Message.HDR_ORIG_MESSAGE_ID);
if (originalID != null) {
- setAnnotation(Message.HDR_ORIGINAL_ADDRESS,
original.getAnnotationString(Message.HDR_ORIGINAL_ADDRESS));
-
- setAnnotation(Message.HDR_ORIG_MESSAGE_ID, originalID);
- } else {
- setAnnotation(Message.HDR_ORIGINAL_ADDRESS, original.getAddress());
+ setBreadcrumb(Message.HDR_ORIGINAL_ADDRESS,
original.getAnnotationString(Message.HDR_ORIGINAL_ADDRESS));
- setAnnotation(Message.HDR_ORIG_MESSAGE_ID, original.getMessageID());
+ setBreadcrumb(Message.HDR_ORIG_MESSAGE_ID, originalID);
}
+ setAnnotation(Message.HDR_ORIGINAL_ADDRESS, original.getAddress());
+
+ setAnnotation(Message.HDR_ORIG_MESSAGE_ID, original.getMessageID());
Review comment:
you probably want to call setBrokerProperty now.
----------------------------------------------------------------
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: 437440)
Time Spent: 2.5h (was: 2h 20m)
> Auto-create DLQ message loss when moving messages between destinations
> ----------------------------------------------------------------------
>
> Key: ARTEMIS-2649
> URL: https://issues.apache.org/jira/browse/ARTEMIS-2649
> Project: ActiveMQ Artemis
> Issue Type: Bug
> Components: Broker
> Affects Versions: 2.12.0
> Environment: Centos 7 container in OKD with Java 8.
> Reporter: Piotr Klimczak
> Priority: Major
> Time Spent: 2.5h
> Remaining Estimate: 0h
>
> [~jbertram], first of all thanks a lot for ARTEMIS-2587 implementation.
> This was a must for me to switch to Artemis.
> In past I have even tried to implement it in Artermis, but having no
> previous experience with it, only with your PR I understood how nicely and
> easily it can be implemented and how much I have overcomplicated it.
> So I am testing 2.12.0 snapshot as I am really interested in work done under
> ARTEMIS-2587.
> I am connecting using open wire protocol using camel-jms component, having
> replaced old AMQ5 with Artermis.
> On failed consumption, I can see queue being created under DLQ address with
> multicast and filter _AMQ_ORIG_ADDRESS = 'some.queue'.
> However it is empty and message is lost.
> Reproduction scenario:
> # Sending message to address A
> # Moving message from A queue to B using web console move function
> # Consuming from B and failing consumption
> Observed state:
> # Queue is being created
> # Message is lost and logs are not indicating anything
> As a result this message being moved from A to be B queue, the header
> "_AMQ_ORIG_ADDRESS" has value "A" instead of "B" and therefore it does not
> match the filter "B" and is getting lost.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
