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.

Bug 124054

Summary: Memory leak in files view
Product: projects Reporter: Jaroslav Tulach <jtulach>
Component: Generic Projects UIAssignee: Milan Kubec <mkubec>
Status: RESOLVED FIXED    
Severity: blocker Keywords: PERFORMANCE, TEST
Priority: P2    
Version: 6.x   
Hardware: All   
OS: All   
Issue Type: DEFECT Exception Reporter:
Bug Depends on:    
Bug Blocks: 121855    

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-[0]->
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