Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 378499 - Table Widget takes too much time to open when opened on big data sets
Summary: Table Widget takes too much time to open when opened on big data sets
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: EMF-Facet (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P2 normal (vote)
Target Milestone: ---   Edit
Assignee: Gregoire Dupe CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-05-04 10:27 EDT by Olivier Remaud CLA
Modified: 2020-05-01 11:26 EDT (History)
2 users (show)

See Also:
gdupe: juno+

Attachments
Proposed patch (8.10 KB, patch)
2012-05-04 10:38 EDT, Olivier Remaud CLA 		no flags Details | Diff
Suggested patch (9.47 KB, text/plain)
2012-05-10 12:07 EDT, Olivier Remaud CLA 		gdupe: iplog+
nicolas.bros: review+
gdupe: review+
Details
Show Obsolete (1) 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 Olivier Remaud CLA 2012-05-04 10:27:34 EDT 
When opened on a large set of objects (rows), the Table widget takes a lot of time to open (more than ten minutes on a powerful computer with less than 40 000 objects)
Comment 1 Olivier Remaud CLA 2012-05-04 10:38:16 EDT 
Created attachment 215065 [details]
Proposed patch

Here is a suggested patch for table opening performance

(a) I, Olivier Remaud, wrote 100% of the code I've provided.
(b) I have the right to contribute the code to Eclipse.
(c) I contribute the content under the EPL.
(d) This contribution contains no Cryptography features.
Comment 2 Olivier Remaud CLA 2012-05-10 12:07:29 EDT 
Created attachment 215414 [details]
Suggested patch

Here is a better patch for table opening performance

(a) I, Olivier Remaud, wrote 100% of the code I've provided.
(b) I have the right to contribute the code to Eclipse.
(c) I contribute the content under the EPL.
(d) This contribution contains no Cryptography features.
Comment 3 Nicolas Bros CLA 2012-05-11 07:56:09 EDT 
Comment on attachment 215414 [details]
Suggested patch

Looks good to me. Thank you for these very needed performance enhancements.
Comment 4 Gregoire Dupe CLA 2012-05-11 08:02:05 EDT 
Comment on attachment 215414 [details]
Suggested patch

This patch is smaller than 250 lines (34 lines).

I've committed this patch.

Committed revision 2205.
Comment 5 Gregoire Dupe CLA 2012-05-23 12:10:10 EDT 
This bug can be marked as fixed.
Comment 6 Gregoire Dupe CLA 2012-05-23 12:31:45 EDT 
Sorry Oliver but this bug has to be closed for the IP log generation.