= Review (NS-134): Bug 1834: Synchronize memory archive with file archive =
|| Author || Jonas ||
|| Moderator || Jonas ||
|| State || Closed ||
== Objectives ==
{{{
Ensure that whenever an external accesses the checksum file the synchronization is performed.
Also, whenever the checksum file is changed the 'lastModifiedChecksumFile' is updated to reflect this change.
}}}
'''Total Time Used (Coding,Documentation,Review)''':

{{{
Time use (Coding,Documentation,Review)
JOLF: 0.4
HBK: 0.1
}}}
'''General comments''':
|| '''Description''' || '''Classification''' || '''Status''' ||


=== Comments on file 'trunk/src/dk/netarkivet/archive/checksum/FileChecksumArchive.java', revision 1207 ===
|| '''Lines''' || '''Description''' || '''Classification''' || '''Status''' ||
|| 159 || Shouldn't this argument be "int minSpaceLeft"? And couldn't this check we replaced with ArgumentNotValid.checkPositive, even though this is a private constructor? || Cosmetic || OK ||
|| 517 || the checksum file? || Cosmetic || OK ||
|| 801 || This should be logged as a warning, not info. || Cosmetic || OK ||