2600
Comment:
|
5306
q
|
Deletions are marked like this. | Additions are marked like this. |
Line 16: | Line 16: |
||General ||Section in Table of contents are not reproduced in the document ||Cosmetic|| || ||Overall Systems Description ||"the first ...." => First we give an overview over the modules, and then go on to describe the database design ||Cosmetic|| || ||Database||Missing fields in jobs table (submitteddate, resubmitted_as_jobId)||Cosmetic|| || ||Database||Is it obvious for everyone what a cross-ref table is?||Cosmetic|| || ||Database||Difficult to see difference between normal tables and cross-ref tables in the schematic|| || || |
||General ||Section in Table of contents are not reproduced in the document ||Cosmetic || || ||Overall Systems Description ||"the first ...." => First we give an overview over the modules, and then go on to describe the database design ||Cosmetic || || ||Database ||Missing fields in jobs table (submitteddate, resubmitted_as_jobId) on schematic ||Cosmetic || || ||Database ||Is it obvious for everyone what a cross-ref table is? ||Cosmetic || || ||Database ||Difficult to see difference between normal tables and cross-ref tables in the schematic ||Cosmetic|| || ||Database||Only possible to access createfullhddb.sql if you have an account on gforge||Cosmetic|| || ||Batch on Bitarchives||Sections "execution and parameters", and "Example of packing and executing a batchjob" does not really belong to the developer manual. Maybe move to user manual. ||Cosmetic|| || ||ViewerProxy||In printer friendly version, uninterpreted html tags areas shown as clear text (e.g. "tt" tag)||Cosmetic|| || ||ViewerProxy||The section headings "Viewerproxy Control resolver", and "Observer resolver" contradicts the contents of these sections||Cosmetic|| || ||Upload of files||Does not belong to developer manual. Belongs to the User manual||Cosmetic|| || ||Harvesting Roundtrip||Reference to table "domainconfigurations" is wrong. The table is named "configurations"||Cosmetic|| || ||Harvesting Roundtrip||"creating new jobs for harvest ...." does not make sense. We are creating jobs on the basis of harvestdefinitions. A set of jobs created on the basis of a harvestdefinition constitutes a harvest||Cosmetic|| || ||Running Heritrix||Reference to a subtle bug in Heritrix. Which bug is that???||Cosmetic|| || ||Harvesting Roundtrip||scheduler executes every minute. Refer to this constant||Cosmetic|| || ||Running Heritrix||l"..log status reports every 20 seconds". Is this a hardwired constant somewhere, or is is wrong?||Cosmetic|| || ||Creating metadata||Note that the metadata-1.arc files are the only arcfiles produced, that have no timestamp included in their name||Cosmetic|| || ||Old jobs||This process is also done after a crawlJob has finished either successfully or otherwise||Cosmetic|| || ||Deduplicates||Describe what deduplication is all aboyt||Cosmetic|| || ||Deduplicates||Add reference(link) to section "Indexing and caching" after subsection starting with "Generation of deduplication indices"||Cosmetic|| || ||Sending Patches||Consider moving this section to the bottom or further up||Cosmetic|| || ||Sending Patches||The link to the twiki page http://netarchive.dk/suite/Process/Create_Tracker_Issue_WithTitle ||Cosmetic|| || ||Settings||Mention that you need to define what local settings-file you are using, if you need to override the default settings. Ddk.netarkivet.settings.file=tests/dk/netarkivet/test-settings.xml|| || || ||Settings||As of version 3.8+|| || || ||Settings||"(also deploy in checks.." (Does not understand?)|| || || ||Settings||"if needed" => If the quickstart requires another value than the default for this setting|| || || ||Settings||"The Deploy manual" => "The Installation manual"|| || || ||Channels|| || || || |
Line 22: | Line 46: |
|| || || || || || || || || || || || || || || || || || || || |
Reviews of Developer Manual devel Please refer to [:Process/OtherReview WithTitle:Other Review Process] for more information New reviews are added at the top
Date: |
2009/03/18 |
Version |
latest version as of 2009/03/17 |
Lines/parts: |
Introduction, Overall Systems Description, Specific functions Description, Settings (section 5,1), Channels (section (5,2),, XML handling by Deploy (section 5.3), Running unit-tests (section 7,2,1) Development using Eclipse or Intellij |
Bugs/Feature requests: |
structure ([https://gforge.statsbiblioteket.dk/tracker/index.php?func=detail&aid=1287&group_id=7&atid=108 FR 1287]) batch ([https://gforge.statsbiblioteket.dk/tracker/index.php?func=detail&aid=1216&group_id=7&atid=105 Bug 1216]), upload, channels ([https://gforge.statsbiblioteket.dk/tracker/index.php?func=detail&aid=1132&group_id=7&atid=105 Bug 1132]) deduplication (task 44 in [:Iteration36TaskList:Iteration 36]) |
Time used: |
SVC: 1 MD, ELZI:? MD, CSR:1 MD |
Line |
Description |
Severity BR Cosmetic, Minor, Major, Showstopper |
Status BR Ok, Postponed, Rejected, Obsolete |
General |
fontsize in printer friendly version |
Cosmetic |
- |
General |
The use of security policies is not described in the manual |
Cosmetic |
|
General |
Fix typos in the manual |
Cosmetic |
|
Introduction |
Find another name for "plugging points" |
Cosmetic |
|
Introduction |
After "or from Subversion repository", add: If you have joined the project at gforge |
Cosmetic |
|
General |
Section in Table of contents are not reproduced in the document |
Cosmetic |
|
Overall Systems Description |
"the first ...." => First we give an overview over the modules, and then go on to describe the database design |
Cosmetic |
|
||Database ||Missing fields in jobs table (submitteddate, resubmitted_as_jobId) on schematic
Cosmetic |
|
||
Database |
Is it obvious for everyone what a cross-ref table is? |
Cosmetic |
|
Database |
Difficult to see difference between normal tables and cross-ref tables in the schematic |
Cosmetic |
|
Database |
Only possible to access createfullhddb.sql if you have an account on gforge |
Cosmetic |
|
Batch on Bitarchives |
Sections "execution and parameters", and "Example of packing and executing a batchjob" does not really belong to the developer manual. Maybe move to user manual. |
Cosmetic |
|
In printer friendly version, uninterpreted html tags areas shown as clear text (e.g. "tt" tag) |
Cosmetic |
|
|
The section headings "Viewerproxy Control resolver", and "Observer resolver" contradicts the contents of these sections |
Cosmetic |
|
|
Upload of files |
Does not belong to developer manual. Belongs to the User manual |
Cosmetic |
|
Harvesting Roundtrip |
Reference to table "domainconfigurations" is wrong. The table is named "configurations" |
Cosmetic |
|
Harvesting Roundtrip |
"creating new jobs for harvest ...." does not make sense. We are creating jobs on the basis of harvestdefinitions. A set of jobs created on the basis of a harvestdefinition constitutes a harvest |
Cosmetic |
|
Running Heritrix |
Reference to a subtle bug in Heritrix. Which bug is that??? |
Cosmetic |
|
Harvesting Roundtrip |
scheduler executes every minute. Refer to this constant |
Cosmetic |
|
Running Heritrix |
l"..log status reports every 20 seconds". Is this a hardwired constant somewhere, or is is wrong? |
Cosmetic |
|
Creating metadata |
Note that the metadata-1.arc files are the only arcfiles produced, that have no timestamp included in their name |
Cosmetic |
|
Old jobs |
This process is also done after a crawlJob has finished either successfully or otherwise |
Cosmetic |
|
Deduplicates |
Describe what deduplication is all aboyt |
Cosmetic |
|
Deduplicates |
Add reference(link) to section "Indexing and caching" after subsection starting with "Generation of deduplication indices" |
Cosmetic |
|
Sending Patches |
Consider moving this section to the bottom or further up |
Cosmetic |
|
Sending Patches |
The link to the twiki page http://netarchive.dk/suite/Process/Create_Tracker_Issue_WithTitle |
Cosmetic |
|
||Settings||Mention that you need to define what local settings-file you are using, if you need to override the default settings. Ddk.netarkivet.settings.file=tests/dk/netarkivet/test-settings.xml|| || ||
Settings |
As of version 3.8+ |
|
|
Settings |
"(also deploy in checks.." (Does not understand?) |
|
|
Settings |
"if needed" => If the quickstart requires another value than the default for this setting |
|
|
Settings |
"The Deploy manual" => "The Installation manual" |
|
|
Channels |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|