Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Manglende sortering af matrikelnumre efter ejerlav #210

Open
klauskjaer opened this issue Oct 13, 2023 · 3 comments
Open

Manglende sortering af matrikelnumre efter ejerlav #210

klauskjaer opened this issue Oct 13, 2023 · 3 comments

Comments

@klauskjaer
Copy link

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
Copy link
Contributor

JesperKock commented Oct 16, 2023

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
Copy link
Author

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
Copy link
Contributor

JesperKock commented Oct 23, 2023

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants