Newspaper digitisation

Review obsolete by now...

Review obsolete by now...

Closing this old review.

Closing this old review.

Ah, tak. Det er en dum copy/paste-fejl af en tidligere dum copy/paste-fejl.

Ah, tak. Det er en dum copy/paste-fejl af en tidligere dum copy/paste-fejl.

Ved nærmere eftertanke, så mindes jeg at opgaven gik på at få tilføjet/tilrettet lidt tests så den ændrede funktionalitet blev sikret.

Ved nærmere eftertanke, så mindes jeg at opgaven gik på at få tilføjet/tilrettet lidt tests så den ændrede funktionalitet blev sikret.

Nu er jeg ikke 100% skarp på hvad ændringerne lige skal gøre (dels fordi jeg lige har været på ferie, men også fordi at jeg ikke lige har været så dybt involveret i opgaven), men som jeg husker det...

Nu er jeg ikke 100% skarp på hvad ændringerne lige skal gøre (dels fordi jeg lige har været på ferie, men også fordi at jeg ikke lige har været så dybt involveret i opgaven), men som jeg husker det drejer det om at filtrere noget logging fra - mere specifikt deepzoom url'erne - altså dem som ender på dzi.

Når jeg kigger på hvad der requestes af mediestream så kommer der en masse requests pr tile der hentes fx:
http://www2.statsbiblioteket.dk/newspaper-stream/6a0015b2-4e4a-4846-9cda-8faa8a4d4f4a?GAM=2.0&CNT=1.1&DeepZoom=/avis-show/symlinks/c/f/9/1/cf917bc4-8503-4d8e-9c45-813bbc497b05.jp2_files/12/6_8.jpg

Jeg ved ikke lige hvor jeg kan hitte en log henne fra webserveren, men bliver samtlige af de her requests ikke også logget? Og vil man have dem logget, eller er det kun det initielle request man er interesserede i?

Som sagt, jeg er ikke helt så godt inde i opgaven, og det ligger meget fjernt hvad den præcist gør (pr logning af requests), men hvis det kun er *.dzi som skal pilles ud af logning ser det ok ud.

'Check that bad resource => a 403' - det kan jeg ikke helt se ske. Den første halvdel af kommentaren giver mere mening da der testes for en HTTP 200. Er det en del af kommentaren der skal stryges?

'Check that bad resource => a 403' - det kan jeg ikke helt se ske. Den første halvdel af kommentaren giver mere mening da der testes for en HTTP 200.
Er det en del af kommentaren der skal stryges?

Add tests
Add tests
Unit test and fix of result collector not being thread safe
Unit test and fix of result collector not being thread safe
Der endte kun med et modul.

Der endte kun med et modul.

Jeg kan ikke huske hvorfor det var nødvendigt med eksplicit angivelse af pom, men det havde den ønskede virkning

Jeg kan ikke huske hvorfor det var nødvendigt med eksplicit angivelse af pom, men det havde den ønskede virkning

"Compares the specified object with this list for equality. Returns true if and only if the specified object is also a list, both lists have the same size, and all corresponding pairs of elements i...

"Compares the specified object with this list for equality. Returns true if and only if the specified object is also a list, both lists have the same size, and all corresponding pairs of elements in the two lists are equal. (Two elements e1 and e2 are equal if (e1==null ? e2==null : e1.equals(e2)).) In other words, two lists are defined to be equal if they contain the same elements in the same order. This definition ensures that the equals method works properly across different implementations of the List interface."

tl;dr - ja

Tager gerne et hurtigt kursus. For nuværende er streng 1-1 overensstemmelse fint.

Tager gerne et hurtigt kursus. For nuværende er streng 1-1 overensstemmelse fint.

Nu er dte et stykke tid sidne, men jeg mener bestemt at @Test på klassen er det samme som @Test på alle metoder.

Nu er dte et stykke tid sidne, men jeg mener bestemt at @Test på klassen er det samme som @Test på alle metoder.

Name clash med java.io.Files.

Name clash med java.io.Files.

Det vil jeg gøre fremover. Det her er leveret til kunden (det er et gammelt review)

Det vil jeg gøre fremover. Det her er leveret til kunden (det er et gammelt review)

Det her kører ude hos brugeren selv. Der er ikke noget fornuftigt sted at sende logfiler hen. Jeg satser derfor på at en fejlmeddelelse alene kan være tilstrækkeligt.

Det her kører ude hos brugeren selv. Der er ikke noget fornuftigt sted at sende logfiler hen. Jeg satser derfor på at en fejlmeddelelse alene kan være tilstrækkeligt.

Det er skrevet som noget af det sidste i tidsnød for at nå at blive færdig til vores kunstige deadlines.

Det er skrevet som noget af det sidste i tidsnød for at nå at blive færdig til vores kunstige deadlines.

Jeg var ikke klar over der var forskellig semantik, men det er jeg så blevet siden koden blev skrevet https://sbforge.org/fisheye/static/nhcjlc/2static/images/wiki/icons/emoticons/wink.gif

Jeg var ikke klar over der var forskellig semantik, men det er jeg så blevet siden koden blev skrevet