Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [hyades-dev] Commiter vote required for M10 check-in

I vote yes for these defects
 
Antony
----- Original Message -----
Sent: Tuesday, May 25, 2004 11:27 PM
Subject: Re: [hyades-dev] Commiter vote required for M10 check-in


Defects are target to be dropped into build tomorrow(Wed May 26) afternoon and regression test will be run as soon as build with the fixes included are available.

Thanks.

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



Harm Sluiman/Toronto/IBM@IBMCA
Sent by: hyades-dev-admin@xxxxxxxxxxx

05/25/2004 06:06 PM

Please respond to
hyades-dev

To
hyades-dev@xxxxxxxxxxx
cc
Subject
Re: [hyades-dev] Commiter vote required for M10 check-in






I vote yes as well, but I would like to know when they will drop into the build and when the full regression test will be run. Adding these 2 bits of information to your format will make a good template for future vote requests.


Thanks for your time.
--------------------------------------------------------------------------
Harm Sluiman, STSM,  
phone:905-413-4032   fax: 4920  
cell: 416-432-9754
mailto:sluiman@xxxxxxxxxx
Admin : Gabrielle Chapman chapmaga@xxxxxxxxxx  Tie: 969-2323


Eugene Chan/Toronto/IBM@IBMCA
Sent by: hyades-dev-admin@xxxxxxxxxxx

05/25/2004 05:05 PM

Please respond to
hyades-dev

To
hyades-dev@xxxxxxxxxxx
cc
Subject
[hyades-dev] Committer vote required for M10 checkin








The following M10 defects require Committer's approval: (I vote YES)


Defect 63508
:
bugzilla_63508 --
Console view has wrong combination of buttons enabled

Rationale
:

There used to be a bug on the execution side preventing the terminated signal from reaching the UI. This has since been fixed; the only remaining thing is to send the platform a signal saying the IProcess has terminated. The console and debug views will update themselves accordingly.


Risk Assessment
:

No API change. The fix is to send the DebugEvent.TERMINATE signal on our IProcess when we get the processExited() event from our internal process.

Low risk change.



Defect 63484
:
bugzilla_63484 --
Profiling types not being notified of launch in some cases

Rationale
:

This fix is to add notification for the profiling types during launch for those launch delegates that were not doing so. This signal is vital for some profiling types in order to perform pre-launch setup. This was missed during implementation of some of our profiling launchers (e.g. Java Applet, Attach - Java Process).


Risk Assessment
:

No API change. The fix is to query the selected profiling set and notify each selected type before performing the launch.

Low risk change.



Defect 63289
:
bugzilla_63289 --
Hosts disappear from the host list in the Profiling Config window after adding new hosts

Rationale
:

This fix updates the host list in preferences when the user adds a host from the launch configuration dialog tab. This was missed during the move to the new launch system.


Risk Assessment
:

No API change. The fix is to update the preferences whenever the user adds a host in the hosts tab.

Low risk change.



Defect 63272
:
bugzilla_63272 --
Profiling tab appears where it shouldn't

Rationale
:

This fix applies to the run/debug mode launch shortcuts in the Profiling and Logging perspective (only from the toolbar launch icon). Because of a platform deficiency, new perspectives cannot inherit existing launch shortcuts; we have to define them again. These re-definitions were pointing to the profiling implementation of the launch shortcuts instead of the run/debug implementations, so it behaved as though the user launched via the profile launch shortcut.


Risk Assessment
:

No API change. Simple fix: switch to the existing run/debug implementation in plugin manifest.

Low risk change.



Defect 62274
:
bugzilla_62274 --
Profiling type attributes should not be stored in launch configuration

Rationale
:

This is a fix for the way profiling sets were incorrectly stored. While most of the contents of the set (e.g. which types are selected) are stored in preferences where they should be, the configuration attributes from the profiling types were stored in the launch configuration. Because of this, any changes to a profiling type that involved a change in attributes affected only one launch configuration. The change should affect all launch configurations that use that set.


Risk Assessment
:

No API change. The fix is to store attributes in preferences with the rest of the set definition, and to no longer store them in launch configurations. Fix verified by the probekit team.

Low/Medium risk change.



Defect 59965
:
bugzilla_59965 --
Show instance level information not taken into account

Rationale
:

This is a fix for the specific scenario where the user selects only the Execution Time Analysis profing type and selects "Show instance level information". In this case, the launcher was sending the wrong options to the rac: TRACE_MODE=full and ALLOCATION_INFORMATION=
none. In this case, we do want allocation information.

Risk Assessment
:

No API change. The fix is to upgrade the value of ALLOCATION_INFORMATION to normal when the user checks the "Show instance level information" box.

Low risk change.


Defect 59371
:
bugzilla_59371 --
Profile launch UI sometimes has a blank line in Profiling...

Rationale
:

This fixes the problem of the blank line showing up in the profiling sets list, and the strange selection behavior (click on one item, another is selected). Although no functionality is missing, this is very annoying to the user.


Risk Assessment
:

No API change. The blank line was caused by circumventing the viewer in some cases and modifying the table widget directly, which confuses the viewer managing the table (viewers are not meant to be used this way). The fix is to do all the work on the table through the viewer. The strange selection behavior was caused by querying the table widget's selection index (same problem). The items were automatically sorted by the viewer, but not in the original list of sets, so we were looking up the wrong index.


Low/Medium risk change.



Thanks for your time.

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


Back to the top