⇤ ← Revision 1 as of 2010-02-09 12:29:33
767
Comment:
|
766
|
Deletions are marked like this. | Additions are marked like this. |
Line 15: | Line 15: |
|| 59 || The names of settings should be defines as public fields in MonitorSettings || Minor || NOTOK || || 95 || add throw ArgumentNotValid clause || Cosmetic || NOTOK || |
|| 59 || The names of settings should be defines as public fields in MonitorSettings || Minor || OK || || 95 || add throw ArgumentNotValid clause || Cosmetic ||Rejected || |
Review (NS-142): FR [#1578] The interval (REREGISTER_DELAY) between apps...
Author |
Henrik Kirk |
Moderator |
Henrik Kirk |
State |
Review |
Objectives
1236: FR [#1578] The interval (REREGISTER_DELAY) between apps re-registering themselves should be a setting
General comments:
Description |
Classification |
Status |
Comments on file 'trunk/src/dk/netarkivet/monitor/distribute/JMSMonitorRegistryClient.java', revision 1236
Lines |
Description |
Classification |
Status |
59 |
The names of settings should be defines as public fields in MonitorSettings |
Minor |
OK |
95 |
add throw ArgumentNotValid clause |
Cosmetic |
Rejected |