Nicholas Clarke

Det ligner c++ syntax og jeg gætter på den laver en .toString(). Jeg har bare brugt dele af den eksisterende implementation.

Det ligner c++ syntax og jeg gætter på den laver en .toString().
Jeg har bare brugt dele af den eksisterende implementation.

Jeg havde en del problemer med at teste for den tomme streng. Det her virkede for mig.

Jeg havde en del problemer med at teste for den tomme streng. Det her virkede for mig.

Tricky. Det udkommenterede er smart når den kører med resten af GUI'en. Og det der kører nu er bedst når den kører stand alone og er nød til at køre på databasen. Så det ville være smart hvis man k...

Tricky. Det udkommenterede er smart når den kører med resten af GUI'en.
Og det der kører nu er bedst når den kører stand alone og er nød til at køre på databasen.
Så det ville være smart hvis man kunne se om den kører standalone eller i GUI.

Det går godt hvis vi får et jobmonitor object. Ellers prøver den igen om 1 minut.

Det går godt hvis vi får et jobmonitor object.
Ellers prøver den igen om 1 minut.

Er exception i java normalt internationaliseret? Hmm...

Er exception i java normalt internationaliseret? Hmm...

Fordømt! Det ved jeg ikke...

Fordømt! Det ved jeg ikke...

Jeg smider en TODO ind. Der kan potentielt komme alt for mange log beskeder som bare er ubrugelige hvis netværk eller h3 er nede.

Jeg smider en TODO ind. Der kan potentielt komme alt for mange log beskeder som bare er ubrugelige hvis netværk eller h3 er nede.

Index filen har en long værdi for alle linie starter i text/crawllog filen. Men starten på den først linie bliver aldrig fundet når routinen nedenunder bruger linefeeds til at opdatere index filen.

Index filen har en long værdi for alle linie starter i text/crawllog filen.
Men starten på den først linie bliver aldrig fundet når routinen nedenunder bruger linefeeds til at opdatere index filen.

Muligvis, jeg tvivler på et job bliver oprettet hvis den ikke kan finde en channel men man kan aldrig vide.

Muligvis, jeg tvivler på et job bliver oprettet hvis den ikke kan finde en channel men man kan aldrig vide.

Ingen meta refresh header i html output.

Ingen meta refresh header i html output.

Javadoc nu tilføjet denne og SearchResult klassen.

Javadoc nu tilføjet denne og SearchResult klassen.

Kunne den godt. Pt. er det bare en copy-paste af H3.

Kunne den godt.
Pt. er det bare en copy-paste af H3.

Det er faktisk blevet ændret men er ikke med i det her review af en eller anden grund.

Det er faktisk blevet ændret men er ikke med i det her review af en eller anden grund.

Jow det kunne man godt argumentere for.

Jow det kunne man godt argumentere for.

Maybe just null@kb.dk.

Maybe just null@kb.dk.

Ikke bevidst. Eclipse begyndte bare at brokke sig over den var unødvendig. Så derfor blev den fjernet. Måske en ændring fra jdk7 til jdk8 fra exception til runtimeexception.

Ikke bevidst. Eclipse begyndte bare at brokke sig over den var unødvendig. Så derfor blev den fjernet. Måske en ændring fra jdk7 til jdk8 fra exception til runtimeexception.

StringBuilder burde gøre det lidt hurtigere og bruge mindre hukommelse. Så det ville kun forbedre læsbarhede en lille smule.

StringBuilder burde gøre det lidt hurtigere og bruge mindre hukommelse.
Så det ville kun forbedre læsbarhede en lille smule.

Yes. Og den har lavet lidt rod i crucible ift. de samme filer flere gange og hvor den ikke helt har styr på de lokale rettelser.

Yes. Og den har lavet lidt rod i crucible ift. de samme filer flere gange og hvor den ikke helt har styr på de lokale rettelser.

NAS-2638: Separate H3 monitor into its own modules and fix some errors.
NAS-2638: Separate H3 monitor into its own modules and fix some errors.
It could stop different threads using the same object and run method. Maybe this was the motivation. Still why would you make more threads using the same runnable object.

It could stop different threads using the same object and run method. Maybe this was the motivation.
Still why would you make more threads using the same runnable object.

I'm pretty sure eav_attributes are not supposed to be used like this in the SQL. But I can be surprised.

I'm pretty sure eav_attributes are not supposed to be used like this in the SQL. But I can be surprised.

This really should be done with JOIN instead.

This really should be done with JOIN instead.

Each domain can have up to the defined amount of attributes. So most likely 3 attributes for each domain that has attributes that are not default. group by should also do it.

Each domain can have up to the defined amount of attributes. So most likely 3 attributes for each domain that has attributes that are not default.
group by should also do it.

Nice catch. Den var et grimt sted før.

Nice catch. Den var et grimt sted før.

Confusion. Deception. World Domination!

Confusion. Deception. World Domination!

Only if you do naughty custom implementations.

Only if you do naughty custom implementations.