Re: [hyades-dev] Committer Approval Requested for 3.0.1.1 Defect Fix -

[Eugene Chan <ewchan@xxxxxxxxxx>]

I vote YES.

--------------------------------------------------------------------
Eugene Chan
IBM Toronto Laboratory
D3/ENW/8200/MKM
Voice:   1-905-413-6102
Email:  ewchan@xxxxxxxxxx

Kent D Siefkes <kdsiefke@xxxxxxxxxx>
Sent by: hyades-dev-admin@xxxxxxxxxxx

10/08/2004 01:30 PM

Please respond to
hyades-dev

To	hyades-dev@xxxxxxxxxxx
cc
Subject	[hyades-dev] Committer Approval Requested for 3.0.1.1 Defect Fix  -  Bugzilla 49076 (75491 duplicate)

Please find below a stop-ship defect that has been approved today by IES
CCB to be included in IES 3.0.1.1. The defect has been reviewed, tested
by producer and consumer, relatively low risk and limited in scope to this
one dialog, and involves no-PII.  (I'm sending out this email for
Joe Toomey). 

A quick description follows: 

75491 - Out of Memory error when using InvokedTestSelectionDialog  
 (this selection dialogue loads the EMF model of each test suite it
displays into memory and never frees it.  Repeated invocations get
slower and slower, until heap memory is exhausted and the JVM crashes,
within 10 invocations if the tests are large) 

49076 - The Add Invocation dialog in the test behavior editor is badly
inefficient  (ditto) 

I'm asking for Hyades committer approval for this defect fix.  No
response within 24 hours indicates silent approval.  Sorry for the
urgency, but IES requires a drop by mid-afternoon on Monday, and given
Monday is a Canadian holiday, we would like to get this dropped today and
approved for adoption by IES on Monday afternoon.

Fixed by keeping record of newly loaded EMF objects by opening the Add
invocation dialog. When dialog is closed, only selected objects are retained
for adding invocation. the rest are explicitly unloaded to enable them
for 
garbage collection.

The above fix has already been made on the head, and will be checked into
3.0.1.1 so that a build can be tested.  If approved, and testing uncovers
no problems,  the fix/build will be included in the next IES 3.0.1.1
build late Monday afternoon. 

Thanks, 
Kent