Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 366943

Summary: [Performance] Editors Memory leaks
Product: [Modeling] Papyrus Reporter: JBregnias <julien.bregnias.ext>
Component: CoreAssignee: Cedric Dumoulin <cedric.dumoulin>
Status: CLOSED FIXED QA Contact:
Severity: major    
Priority: P1 CC: cedric.dumoulin, faure.tristan, mathieu.velten, raphael.faudou
Version: 0.8.1   
Target Milestone: ---   
Hardware: All   
OS: All   
Whiteboard:
Bug Depends on: 356837, 368704, 371604, 371653, 384145, 384169    
Bug Blocks:    
Attachments:
Description Flags
the workaround for the patch
none
the result of the memory leak analysis
none
mylyn/context/zip none

Description JBregnias CLA 2011-12-16 08:59:50 EST 
Using papyrus editors for one day causes that the platform memory is bigger than 800Mo, even if memory is necessary from others applications (not a jvm problem).
Comment 1 Tristan Faure CLA 2011-12-30 09:29:41 EST 
Created attachment 208870 [details]
the workaround for the patch
Comment 2 Tristan Faure CLA 2011-12-30 09:34:25 EST 
Created attachment 208872 [details]
the result of the memory leak analysis
Comment 3 Tristan Faure CLA 2011-12-30 09:36:02 EST 
The problem comes from a dispose problem
for developers :
when a tab is closed the different papyrus parts (inheriting from AbstractPart) are correctly updated and the closed one is disposed.

when the "close current file" action is performed the different dispose methods are not closed because there is no links between the CTabItem (correctly disposed) and the diagram editor.

the patch i attach provides a simple work around but it does not completely fix the memory leak.
Cedric do you have any clue to provide a better way to fix this ?

I attache too, one screenshot corresponding to the leak analysis the work around applied to the sources

we are very concerned about this bug and I set a priority P1 to it.
Comment 4 Tristan Faure CLA 2012-02-29 10:37:42 EST 
Created attachment 211811 [details]
mylyn/context/zip

context of commit
Comment 5 Tristan Faure CLA 2012-02-29 10:37:53 EST 
commited Cedric Dumoulin modifications to the 0.8.X branch
Comment 6 Mathieu Velten CLA 2012-08-24 04:35:26 EDT 
memory leak in the DiagramEventBroker : some editpolicies are registering themselves as listeners but never unregistered in the deactivate.
Comment 7 Mathieu Velten CLA 2012-08-24 05:27:33 EDT 
fixed on 0.8.X-EYY and trunk.