Greetings, folks:

This Wednesday (04.11.2005) at noon in POST 307, we will be reviewing
classes from hackyIssue involving the new reducer and supporting
classes.
1) Package org.hackystat.stdext.issue.reducer
2) Some classes in the org.hackystat.stdext.issue.sdt package:
   org.hackystat.stdext.issue.sdt.IssuePriorityManager
   org.hackystat.stdext.issue.sdt.TestIssuePriorityManager
   org.hackystat.stdext.issue.sdt.IssueStatusManager
   org.hackystat.stdext.issue.sdt.TestIssueStatusManager
   org.hackystat.stdext.issue.sdt.IssueTypeManager
Note: a test class does not exist for IssueTypeManager since it's
methods are very simple and straightforward.

Please make sure you install the latest version of Jupiter and Jupiter
sensor. Otherwise you might not send the correct data to the server.
----------------------------------

To perform this review:
1) Install and/or update (a) the Eclipse Jupiter plug-in and (b) The
Hackystat Jupiter sensor.
2) Checkout/Update the hackyIssue module from CVS. hackyIssue depends
on [hackyBuild, hackyKernel, hackyReport, hackyStdExt, hackyTelemetry].
Please check out any of these modules if you do not have them.
3) Select "IssueReducer" as the ReviewID.
----------------------------------

Please focus your attention on:
org.hackystat.stdext.issue.reducer

----------------------------------

Here are some questions to consider:

1) Is the calculation accurate?

Especially review the formula used to calculate the data points for
the 'rate of closure' modes.

2) Could the class be structured better?

The reducer is modularized into a bunch of different methods. Please
review whether you agree with it's design. Also, please give any
comments on the use of the manager classes.


Thank you for reviewing the code!

Burt

-------------------------------------------------------------
Supplemental Links and Resources:

The Hackystat Code Review Guidelines:
* http://hackydev.ics.hawaii.edu/hackyDevSite/doc/Review.html

The Elements of Hacky Style:
* http://hackydev.ics.hawaii.edu/hackyDevSite/doc/EHS.html

Jupiter Code Review Tool documentation:
* http://csdl.ics.hawaii.edu/Tools/Jupiter/Core/doc/UsersGuide.html

Hackystat Jupiter Sensor documentation:
* http://hackystat.ics.hawaii.edu/hackystat/docbook/apas15.html

Reply via email to