SDFIdk / gsearch

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Manglende sortering af matrikelnumre efter ejerlav

klauskjaer opened this issue · comments

Hvis man søger med et matrikelnummer, ser det ud til, at resultatlisten leverer resultater i random rækkefølge.
Rækkefølgen skifter fra kald til kald, selvom det er samme parametre.
Fx:
https://api.dataforsyningen.dk/rest/gsearch/v1.0/matrikel?q=1a&limit=100

Resultatlisten bør være sorteret alfabetisk efter ejerlavsnavn, da det typisk vil være ejerlavsnavnet man vil præsentere i søgelisten.

Her er fx et par skærmdumps fra et par af vores løsninger:
GeoDK
image

Matriklen.dk
image

Hvis en query i gsearch returnere for mange resulter så bliver der lavet et simplificeret svar for at give et svar hurtigere.

At tilføje en ORDER BYejerlavmatrikel betyder at den specifikke søgning 1a vil det tage ca. 5 sek. før der kommer et svar.

Vi undrer os over, at denne order by kan tage 5 sek. - men hvis det er konsekvensen af jeres implementering vil vi foretrække at det undlades...

Det tager 5 sek fordi søgningen 1a leder efter et match i kolonnerne matrikelnummer og ejerlavsnavn. Hvis API'en i stedet søgte efter et match kun i matrikelnummer med en ORDER BYmatrikelnummer ville det tage 40-50ms. Men det betyder at en søgning på f.eks. by ikke vil give nogle resultater hvilke det gør nu.

Jeg har holdt issuet'et åbent fordi jeg er sådanset enig i at der burde være en sortering på resultaterne. Men pt. kan vi ikke ændre noget uden enten at væsentlig ændre noget funktionalitet eller reducere svartiderne.