[
https://issues.apache.org/jira/browse/ARTEMIS-3067?focusedWorklogId=539994&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-539994
]
ASF GitHub Bot logged work on ARTEMIS-3067:
-------------------------------------------
Author: ASF GitHub Bot
Created on: 22/Jan/21 06:45
Start Date: 22/Jan/21 06:45
Worklog Time Spent: 10m
Work Description: franz1981 commented on a change in pull request #3407:
URL: https://github.com/apache/activemq-artemis/pull/3407#discussion_r562416623
##########
File path:
artemis-server/src/main/java/org/apache/activemq/artemis/core/server/impl/QueueImpl.java
##########
@@ -3716,6 +3730,16 @@ private boolean deliver(final MessageReference ref) {
}
}
+ private void accountForChangeInMemoryEstimate(final MessageReference ref,
final int existingMemoryEstimate) {
+ final int delta = ref.getMessageMemoryEstimate() -
existingMemoryEstimate;
+ if (delta > 0) {
+ PagingStore pageStore = ref.getOwner();
+ if (pageStore != null) {
+ pageStore.addSize(delta);
Review comment:
2 observations:
1. what happen to core messages here?
2. this method could become static
----------------------------------------------------------------
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: 539994)
Time Spent: 1.5h (was: 1h 20m)
> AMQP applicationProperties are not part of the memoryEstimate
> -------------------------------------------------------------
>
> Key: ARTEMIS-3067
> URL: https://issues.apache.org/jira/browse/ARTEMIS-3067
> Project: ActiveMQ Artemis
> Issue Type: Bug
> Components: AMQP
> Affects Versions: 2.16.0
> Reporter: Gary Tully
> Assignee: Gary Tully
> Priority: Major
> Fix For: 2.17.0
>
> Time Spent: 1.5h
> Remaining Estimate: 0h
>
> With significant data in application properties, the decoded properties can
> consume memory that is not tracked for paging purposes and can lead to
> unexpected OOM.
> Duplicate detection is one cause of decoding. Use of selectors is another.
> Otherwise they are left intact by the broker and just routed in their raw
> format.
> When they are decoded, for whatever reason, we need to account for them is
> some way such that paging can kick in as expected.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
