You are viewing a plain text version of this content. The canonical link for it is here.
Posted to gitbox@activemq.apache.org by "essobedo (via GitHub)" <gi...@apache.org> on 2023/09/08 15:51:35 UTC

[GitHub] [activemq] essobedo opened a new pull request, #1051: [AMQ-9255] Initialize the transient field of the class Message

essobedo opened a new pull request, #1051:
URL: https://github.com/apache/activemq/pull/1051

   Fixes https://issues.apache.org/jira/browse/AMQ-9255
   
   ## Motivation
   
   Persistent messages with an expired TTL are not moved to the DLQ when the HTTP protocol is used as transport. 
   
   ## Modifications:
   
   * Initialize the field `Message#processAsExpired` at deserialization
   * Add a unit test to reproduce the bug


-- 
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.

To unsubscribe, e-mail: gitbox-unsubscribe@activemq.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [AMQ-9255] Initialize the transient field of the class Message [activemq]

Posted by "yijut2 (via GitHub)" <gi...@apache.org>.
yijut2 commented on PR #1051:
URL: https://github.com/apache/activemq/pull/1051#issuecomment-1817990286

   Hi, 
   
   I met an issue when using Nondex, a tool for detecting and debugging wrong assumptions on under-determined Java APIs. (https://github.com/TestingResearchIllinois/NonDex) When I run the detect flakiness command on the AMQ9255Test, sometimes I will get this error:
   `[ERROR] Errors: 
   [ERROR]   AMQ9255Test.testExpiredMessages:146 ยป JMS Could not post command: MessageAck {commandId = 5, responseRequired = false, ackType = 2, consumerId = ID:fa23-cs527-073.cs.illinois.edu-37869-1700429033032-5:2:1:1, firstMessageId = ID:fa23-cs527-073.cs.illinois.edu-37869-1700429033032-5:1:1:1:1, lastMessageId = ID:fa23-cs527-073.cs.illinois.edu-37869-1700429033032-5:1:1:1:1, destination = queue://ActiveMQ.DLQ, transactionId = null, messageCount = 1, poisonCause = null} due to: org.apache.http.NoHttpResponseException: localhost:8161 failed to respond
   [INFO] 
   [ERROR] Tests run: 1, Failures: 0, Errors: 1, Skipped: 0`
   
   This error won't appear everytime, but just sometimes. Did you have an idea where the problem is? Thank you in advance.


-- 
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.

To unsubscribe, e-mail: gitbox-unsubscribe@activemq.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [AMQ-9255] Initialize the transient field of the class Message [activemq]

Posted by "cshannon (via GitHub)" <gi...@apache.org>.
cshannon merged PR #1051:
URL: https://github.com/apache/activemq/pull/1051


-- 
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.

To unsubscribe, e-mail: gitbox-unsubscribe@activemq.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org