SDFIdk / gsearch

MIT License
5 stars 0 forks source link

Manglende sortering af matrikelnumre efter ejerlav #210

Open klauskjaer opened 1 year ago

klauskjaer commented 1 year ago

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
JesperKock commented 1 year ago

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.

klauskjaer commented 1 year ago

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...

JesperKock commented 1 year ago

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.