[
https://issues.apache.org/jira/browse/ARTEMIS-2188?focusedWorklogId=186603&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-186603
]
ASF GitHub Bot logged work on ARTEMIS-2188:
-------------------------------------------
Author: ASF GitHub Bot
Created on: 17/Jan/19 22:52
Start Date: 17/Jan/19 22:52
Worklog Time Spent: 10m
Work Description: asfgit commented on pull request #2446: ARTEMIS-2188
fix address size leak caused by large page message
URL: https://github.com/apache/activemq-artemis/pull/2446
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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: 186603)
Time Spent: 10m
Remaining Estimate: 0h
> Paged large message results in pagingStore sizeInBytes leak when the page is
> evicted
> ------------------------------------------------------------------------------------
>
> Key: ARTEMIS-2188
> URL: https://issues.apache.org/jira/browse/ARTEMIS-2188
> Project: ActiveMQ Artemis
> Issue Type: Bug
> Components: Broker
> Reporter: yangwei
> Priority: Major
> Time Spent: 10m
> Remaining Estimate: 0h
>
> When we write a paged large message,
> addLiveMessage->incrementDelayDeletionCount->
> incrementRefCount->onDurableUp is called where pagingstore size and global
> size is added.
> The page where the large message resides maybe evicted from cache.
> When the page is deleted, LargeServerMessageImpl::decrementDelayDeletionCount
> is called the last time but the large message is not the original one which
> means context is null and nonDurableDown is not called resulting in
> pagingstore size and global size not properly subtracted.
> Writing large paged message should be handled like regular paged message. The
> large paged message doesn't account for the memory size, we don't need to
> call PagingStoreImpl::onDurableUp.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
