Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 299797 - [typesystem.emf] Use of eAllContents in getKnownTypes yields massive overhead on large metamodels
Summary: [typesystem.emf] Use of eAllContents in getKnownTypes yields massive overhead...
Status: CLOSED FIXED
Alias: None
Product: M2T
Classification: Modeling
Component: Xpand (show other bugs)
Version: 0.8.0   Edit
Hardware: PC All
: P3 minor (vote)
Target Milestone: M5   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-01-15 12:43 EST by Erhard Weinell CLA
Modified: 2013-02-21 08:12 EST (History)
1 user (show)

See Also:
sven.efftinge: helios+

Attachments
Replaces eAllContents with a more specific implementation (2.22 KB, patch)
2010-01-15 12:45 EST, Erhard Weinell CLA 		sven.efftinge: iplog+
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 Erhard Weinell CLA 2010-01-15 12:43:35 EST 
Using eAllContents to determine all known types is very expensive as many non-classifier elements are iterated

Steps to reproduce:
 * register a large metamodel (e.g. UML2)
 * invoke getKnownTypes
 
 Solution: Iterate through sub-packages and contained classifiers only.
Comment 1 Erhard Weinell CLA 2010-01-15 12:45:19 EST 
Created attachment 156253 [details]
Replaces eAllContents with a more specific implementation
Comment 2 Sven Efftinge CLA 2010-01-28 03:15:39 EST 
patch applied. Thanks!
Comment 3 Karsten Thoms CLA 2013-02-21 08:12:56 EST 
Bug resolved before Xpand 1.2 release date => Closing