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 51506 - ValuesObject leak
Summary: ValuesObject leak
Status: CLOSED FIXED
Alias: None
Product: java
Classification: Unclassified
Component: Unsupported (show other bugs)
Version: 4.x
Hardware: All All
: P1 blocker (vote)
Assignee: issues@java
URL:
Keywords: PERFORMANCE
Depends on:
Blocks:
 
Reported: 2004-11-15 09:58 UTC by Petr Nejedly
Modified: 2006-03-24 09:41 UTC (History)
3 users (show)

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Petr Nejedly 2004-11-15 09:58:19 UTC
Scenario from issue 51384.

I've found extraordinary increase in number of
Integer instances and HashMap$Entry
instances (tens of thousands) and traced it to the
MdrStorage singleton.
MdrStorage keeps track of all ever created
instances of ValuesObject.
There is a way to register new instance of
ValuesObject (one ValuesObject instance
per one storageId), but there is no way of
unregistering it, so I found >300 instances
of ValuesObject registered. Each ValuesObject
keeps quite large list of values and a map
which maps values to integers. These objects are
kept even after closing all related projects.
Comment 1 Martin Matula 2004-11-15 10:08:04 UTC
Fixed.

Checking in src/org/netbeans/mdr/storagemodel/MdrStorage.java;
/cvs/mdr/src/org/netbeans/mdr/storagemodel/MdrStorage.java,v  <-- 
MdrStorage.java
new revision: 1.70; previous revision: 1.69
done
Comment 2 Petr Nejedly 2004-11-15 15:06:15 UTC
I can confirm that ValuesObject instances get properly freed and most
of Integer instances as well in current trunk sources.
Comment 3 Jan Chalupa 2004-11-18 16:05:46 UTC
Please prepare a fix for 4.0. Thanks.
Comment 4 Martin Matula 2004-11-20 00:04:49 UTC
Fixed in release40.

Checking in src/org/netbeans/mdr/storagemodel/MdrStorage.java;
/cvs/mdr/src/org/netbeans/mdr/storagemodel/MdrStorage.java,v  <-- 
MdrStorage.java
new revision: 1.68.2.1; previous revision: 1.68
done
Comment 5 Petr Nejedly 2004-11-22 13:24:35 UTC
Verified in release40 too.