qpid-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Robbie Gemmell (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (QPID-3387) QueueEntries can hold the Subscriptions which rejected them in memory after they are closed
Date Wed, 03 Aug 2011 21:47:27 GMT

     [ https://issues.apache.org/jira/browse/QPID-3387?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Robbie Gemmell updated QPID-3387:
---------------------------------

    Description: 
QueueEntries can hold Subscriptions which rejected them in memory after they are closed, due
to the implementation of the isRejectedBy() method. This is further compounded by the Subscriptions
having a reference to the channel/session they belong to, which in the case of a 0-10 ServerSession
can be an extremely heavy weight object in the wrong conditions. The isRejectedBy() implementation
should be updated to function without requiring a reference to the Subscription itself, thus
preventing the retention leak.

I have had a simple patch for this for a while, its just waiting for QPID-3386 being done
(which it almost is, will get it committed shortly).

  was:
QueueEntries can hold Subscriptions which rejected them in memory after they are closed, due
to the implementation of the isRejectedBy() method. This is further compounded by the Subscriptions
having a reference to the channel/session they belong to, which in the case of a 0-10 ServerSession
can be an extremely heavy weight object in the wrong conditions. The isRejectedBy() implementation
should be updated to function without requiring a reference to the Subscription itself, thus
preventing the retention leak.

I have had a simple patch for this for a while, its just waiting for QPID-3336 being done
(which it almost is, will get it committed shortly).


> QueueEntries can hold the Subscriptions which rejected them in memory after they are
closed
> -------------------------------------------------------------------------------------------
>
>                 Key: QPID-3387
>                 URL: https://issues.apache.org/jira/browse/QPID-3387
>             Project: Qpid
>          Issue Type: Bug
>          Components: Java Broker
>    Affects Versions: 0.5, 0.6, 0.7, 0.8, 0.9, 0.10, 0.11, 0.12
>            Reporter: Robbie Gemmell
>            Assignee: Robbie Gemmell
>             Fix For: 0.13
>
>
> QueueEntries can hold Subscriptions which rejected them in memory after they are closed,
due to the implementation of the isRejectedBy() method. This is further compounded by the
Subscriptions having a reference to the channel/session they belong to, which in the case
of a 0-10 ServerSession can be an extremely heavy weight object in the wrong conditions. The
isRejectedBy() implementation should be updated to function without requiring a reference
to the Subscription itself, thus preventing the retention leak.
> I have had a simple patch for this for a while, its just waiting for QPID-3386 being
done (which it almost is, will get it committed shortly).

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

---------------------------------------------------------------------
Apache Qpid - AMQP Messaging Implementation
Project:      http://qpid.apache.org
Use/Interact: mailto:dev-subscribe@qpid.apache.org


Mime
View raw message