[
https://issues.apache.org/jira/browse/QPID-1385?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12642470#action_12642470
]
Martin Ritchie commented on QPID-1385:
--------------------------------------
During the refactor it was also noted that there was a code path that simply
logged 'Dropping Message' but we never actually discarded the message in that
path.
This would lead to a memory leak if we did not discard the message as the
reference count would be wrong.
So I added the message.discard() so that it would mirror the existing message
drop a few lines down and a test to cover the case.
> CommitRollbackTest.testSend2ThenRollback failure from build system
> ------------------------------------------------------------------
>
> Key: QPID-1385
> URL: https://issues.apache.org/jira/browse/QPID-1385
> Project: Qpid
> Issue Type: Bug
> Components: Java Tests
> Affects Versions: M3, M4
> Reporter: Martin Ritchie
> Assignee: Rob Godfrey
> Fix For: M4
>
> Attachments:
> TEST-org.apache.qpid.test.unit.transacted.CommitRollbackTest.testSend2ThenRollback.out
>
>
> Summary:
> The following failure occurred last night:
> testSend2ThenRollback
> org.apache.qpid.test.unit.transacted.CommitRollbackTest
> org.apache.qpid.client.JMSAMQException: Fail-over interrupted commit. Status
> of the commit is uncertain.
> at org.apache.qpid.client.AMQSession.commit(AMQSession.java:740)
> at
> org.apache.qpid.test.unit.transacted.CommitRollbackTest.testSend2ThenRollback(CommitRollbackTest.java:370)
>
> at org.apache.qpid.test.utils.QpidTestCase.runBare(QpidTestCase.java:186)
> at org.apache.qpid.test.utils.QpidTestCase.run(QpidTestCase.java:220)
> Attached is the full output from the test.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
