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 - Memory leak in files view
Summary: Memory leak in files view
Status: RESOLVED FIXED
Alias: None
Product: projects
Classification: Unclassified
Component: Generic Projects UI (show other bugs)
Version: 6.x
Hardware: All All
: P2 blocker (vote)
Assignee: Milan Kubec
URL:
Keywords: PERFORMANCE, TEST
Depends on:
Blocks: 121855
  Show dependency tree
 
Reported: 2007-12-14 22:56 UTC by Jaroslav Tulach
Modified: 2008-01-14 14:25 UTC (History)
0 users

See Also:
Issue Type: DEFECT
Exception Reporter:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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