This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.
|Summary:||Memory leak in files view|
|Product:||projects||Reporter:||Jaroslav Tulach <jtulach>|
|Component:||Generic Projects UI||Assignee:||Milan Kubec <mkubec>|
|Issue Type:||DEFECT||Exception Reporter:|
|Bug Depends on:|
Description Jaroslav Tulach 2007-12-14 22:56:28 UTC
junit.framework.AssertionFailedError: Checking if all projects are really garbage collected Project: private static final org.openide.loaders.DataFilter org.netbeans.modules.project.ui.PhysicalView$GroupNode.VISIBILITY_QUERY_FILTER-> org.netbeans.modules.project.ui.PhysicalView$VisibilityQueryDataFilter@1c1cda4-changeSupport-> org.openide.util.ChangeSupport@1656e9e-listeners-> java.util.concurrent.CopyOnWriteArrayList@cf1b5c-array-> [Ljava.lang.Object;@17ba8f8--> org.openide.loaders.FolderChildren@fc1399-parent-> org.netbeans.modules.project.ui.PhysicalView$GroupNode@144d1b6-project-> org.netbeans.modules.java.j2seproject.J2SEProject@1d2d6f7
Comment 1 Jaroslav Tulach 2008-01-11 09:40:40 UTC
the performance team would like to ask your teams for help with making memory leak tests part of the commit validation. One of the main performance goals for 6.1 is to fix memory leaks in main use cases and prevent regressions in future. As the first step the performance team plans to extend the commit validation tests by checking whether certain important data structures are freed from memory after the tests finish. All leaks found so far are filed under 121855 umbrella issue: http://openide.netbeans.org/issues/showdependencytree.cgi?id=121855 As you can see, the number of issues is not high (14 at this moment, some of them already fixed). We need to fix the rest - and fix leaks that appear in the future preferentially (as part of commit validation). Thus we will make the issues of P2 priority.
Comment 2 Jan Lahoda 2008-01-11 11:30:03 UTC
I am a bit confused - the last comment suggests this issue should have priority P2, but the current priority is P3?
Comment 3 Milan Kubec 2008-01-11 11:47:08 UTC
Thanks for catching it. Let's make it P2. There's never enough of P2s.
Comment 4 Milan Kubec 2008-01-14 14:25:47 UTC
Fixed. There is no need to keep reference to project in GroupNode, it's not used anywhere. Checking in PhysicalView.java; /cvs/projects/projectui/src/org/netbeans/modules/project/ui/PhysicalView.java,v <-- PhysicalView.java new revision: 1.22; previous revision: 1.21 done