[
https://issues.apache.org/jira/browse/ARTEMIS-2200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16745588#comment-16745588
]
clebert suconic commented on ARTEMIS-2200:
------------------------------------------
The real issue here is the checkValidIDs is performed before calling the
execute, while the actual delete happens inside the executor. This could have
issues if running from multi-threads.
I fixed the real issue on the journal. While Franz is avoiding multiple calls
as well.
> NPE while dropping/failing large messages on paging
> ---------------------------------------------------
>
> Key: ARTEMIS-2200
> URL: https://issues.apache.org/jira/browse/ARTEMIS-2200
> Project: ActiveMQ Artemis
> Issue Type: Bug
> Affects Versions: 2.7.0, 2.6.3
> Reporter: Francesco Nigro
> Assignee: Francesco Nigro
> Priority: Minor
> Fix For: 2.7.0
>
> Time Spent: 1.5h
> Remaining Estimate: 0h
>
> Due to a race condition while handling deletion (ie confirmation) of large
> messages dropped/failed due to paging, the Artemis journal try to delete the
> same records twice, throwing NPE while doing it.
> In addition, given that the NPE is unexpected, the broker is not cleaning up
> the pending stores on the OperationContext, leading to some leaking
> OperationContextImpl.TaskHolder instances.
> It is happening with any protocols ie Core, OpenWire, AMQP.
> Are not known yet other bad effects beyond the NPE and the leak.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
