Bug 309156

Summary: too many event object created in event queue
Product: [WebTools] JSDT Reporter: Michael Rennie <Michael_Rennie>
Component: DebugAssignee: Michael Rennie <Michael_Rennie>
Status: RESOLVED WONTFIX QA Contact: Simon Kaegi <simon_kaegi>
Severity: major    
Priority: P2 CC: thatnitind
Version: 3.2Keywords: performance
Target Milestone: ---   
Hardware: All   
OS: All   
Whiteboard:

Description Michael Rennie CLA 2010-04-14 12:08:05 EDT
code from HEAD

Testing new events for bug 308330 I found that we create N event object when there are N requests for the event. We do this because we pas back the request object in the event. 

We should investigate not doing this, we should be creating 1 event and broadcasting it to all requestors (only place one event in the queue). The reasons for this are many, mostly:

1. smaller event queue size
2. smaller event / event packet size
3. less work sending events (less to iterate over)
4. faster event creation - we don't have to iterate over all requestors creating objects

etc.
Comment 1 Michael Rennie CLA 2010-04-23 09:55:16 EDT
This is not going to happen for M7
Comment 2 Michael Rennie CLA 2010-08-17 02:00:57 EDT
we looked at this in M1 and decided to leave it as is, one event for one request is fine, and a good way to avoid broadcasting events when we shouldn't be.

Marking wontfix (before I forget again)