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 171657

Summary:	Expand All usages result may run out of memory
Product:	java	Reporter:	Jan Pokorsky <jpokorsky>
Component:	Refactoring	Assignee:	Jan Pokorsky <jpokorsky>
Status:	RESOLVED FIXED
Severity:	blocker	CC:	pflaska
Priority:	P2	Keywords:	PERFORMANCE
Version:	6.x
Hardware:	All
OS:	All
Issue Type:	DEFECT	Exception Reporter:
Bug Depends on:
Bug Blocks:	171188

Description Jan Pokorsky 2009-09-07 18:10:41 UTC

The DBObject query finishes and releases memory properly now. The result appears as a collapsed project node. But when
you choose Expand All action it is horribly slow, blocks EDT and ends up with OOME quite often.

It is caused by loading documents to compute usage's line when nodes are created in CheckRenderer
(PositionBounds.getBegin().getLine()). Since the line information is used only for node's tooltip it should be possible
to postpone it to the moment when user hovers the mouse over the node and if possible not to load the document at all.

Comment 1 Jan Pokorsky 2009-09-08 15:41:29 UTC

fixed as jet-main/rev/954fd1877405

My measure of the DBObject query (6373 occurrences) shows 90% improvement for the Expand All action. I had to increase
the heap limit to measure something before the patch. Otherwise I got OOMEs.

Comment 2 Quality Engineering 2009-09-11 21:46:34 UTC

Integrated into 'main-golden', will be available in build *200909111401* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress)
Changeset: http://hg.netbeans.org/main-golden/rev/954fd1877405
User: Jan Pokorsky <jpokorsky@netbeans.org>
Log: #171657: do not load document to render CheckNode