766
Comment:
|
← Revision 3 as of 2010-08-16 10:24:46 ⇥
768
converted to 1.6 markup
|
Deletions are marked like this. | Additions are marked like this. |
Line 1: | Line 1: |
= Review (NS-142): FR [#1578] The interval (REREGISTER_DELAY) between apps... = | = Review (NS-142): FR [[#1578]] The interval (REREGISTER_DELAY) between apps... = |
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 |