[ 
https://issues.apache.org/jira/browse/CAMEL-5058?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13223140#comment-13223140
 ] 

Claus Ibsen commented on CAMEL-5058:
------------------------------------

Notice ideally the inflight registry would just track for routes only. But the 
producer template uses an UoW as well for some reason we had to do a while ago. 
So it will also enlist itself in the inflight registry (well its in fact also 
an inflight message you may say).

So we can either ensure to remove from the inflight map if it become zero as 
you suggest. This may require a bit of locking, as you need to remove and 
compare.

Alternative, as the map, does not need to be removed in real time, we can have 
a background thread, doing the cleanup in a scheduled fashion.
                
> Bug: Unique Endpoints Leaking in DefaultInflightRepository
> ----------------------------------------------------------
>
>                 Key: CAMEL-5058
>                 URL: https://issues.apache.org/jira/browse/CAMEL-5058
>             Project: Camel
>          Issue Type: Bug
>          Components: camel-core
>    Affects Versions: 2.9.0
>            Reporter: Zach Calvert
>            Assignee: Claus Ibsen
>             Fix For: 2.10.0, 2.9.2
>
>   Original Estimate: 24h
>  Remaining Estimate: 24h
>
> If you have an endpoint protocol which uses unique URIs you will leak Strings 
> in the HashMap stored in the DefaultInflightRepository 
> (org.apache.camel.impl.DefaultInflightRepository)
> It seems there is a reference counting scheme in place, but it doesn't do a 
> remove until the "stop" method is called to shut the system down.  We are 
> running XMPP endpoints, which use a protocol like 
> xmpp://someaccount@domain/password?to=someOtherAccount
> When there are 10 million accounts, not all of which are active, but all of 
> which may message at some time or another, no references are removed to the 
> endpointCount.
> When the count becomes 0, the reference should be removed and the size method 
> will still return the appropriate result.
> Please be careful in the implementation to synchronize on some object 
> (perhaps the AtomicInteger) reflecting a read/write lock on the endpoint 
> count modification.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to