You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@synapse.apache.org by "Hiranya Jayathilaka (JIRA)" <ji...@apache.org> on 2010/10/18 03:19:26 UTC

[jira] Issue Comment Edited: (SYNAPSE-701) Remove the finalize method of the TemporaryData.java in commons module of synapse

    [ https://issues.apache.org/jira/browse/SYNAPSE-701?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12921935#action_12921935 ] 

Hiranya Jayathilaka edited comment on SYNAPSE-701 at 10/17/10 9:18 PM:
-----------------------------------------------------------------------

Andreas,

Under what circumstances these files may not get cleaned up? I don't think the Synapse TemporaryData class is affected by the Axis2 architecture flaw you have described in the blog post. This class is a pretty independent implementation which resides in Synapse. So can't we fix Synapse to properly remove the temp files by calling the release method in TemporaryData class?



      was (Author: hiranya):
    Andreas,

Under what circumstances these files may not get cleaned up? Can we fix those instances to properly remove the temp files by calling the release method?


  
> Remove the finalize method of the TemporaryData.java in commons module of synapse
> ---------------------------------------------------------------------------------
>
>                 Key: SYNAPSE-701
>                 URL: https://issues.apache.org/jira/browse/SYNAPSE-701
>             Project: Synapse
>          Issue Type: Bug
>            Reporter: Supun Kamburugamuva
>            Assignee: Supun Kamburugamuva
>
> A finalize method can cause delayed garbage collection as well as it doesn't guarantee a clean. This can lead to potential Out of Memory errors if these classes are used very frequently. Also this method is redundant because this object has a explicit release method. So I suggest we remove this redundant method.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@synapse.apache.org
For additional commands, e-mail: dev-help@synapse.apache.org