Bug 226319 - [misc] ContentAssistTestSuite uses lots of memory
Summary: [misc] ContentAssistTestSuite uses lots of memory
Status: RESOLVED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Text (show other bugs)
Version: 3.4   Edit
Hardware: PC Windows XP
: P4 normal (vote)
Target Milestone: 3.4 M7   Edit
Assignee: JDT-Text-Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords: test
Depends on:
Blocks:
 
Reported: 2008-04-09 13:17 EDT by Martin Aeschlimann CLA
Modified: 2008-04-11 07:22 EDT (History)
1 user (show)

See Also:

Attachments
patch (1.55 KB, patch)
2008-04-10 11:42 EDT, Martin Aeschlimann CLA 		no flags Details | Diff
View All 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 Martin Aeschlimann CLA 2008-04-09 13:17:27 EDT 
20080409

I run the ContentAssistTestSuite and ran out of memory.

When inspecting with yourkit I saw that half of the memory (30 MB) is used in the UISynchronizer.messages; that's the queue of async-execs to be executed.

The fix would be to run the display loop between tests until empty. This would also make sure that the next test can start in a cleaner state.
Comment 1 Dani Megert CLA 2008-04-10 02:20:44 EDT 
Sorry now time to work on this for now, but patch is welcome. These tests run for years without ever seeing an OOME.
Comment 2 Martin Aeschlimann CLA 2008-04-10 11:42:53 EDT 
Created attachment 95545 [details]
patch

I tracked down the runnables that were in the list and was also able to make some fixes in our land. There are now much fewer runnable around as before.

I still recommend to apply the patch.
Comment 3 Martin Aeschlimann CLA 2008-04-11 07:22:07 EDT 
fixed > 20080411