NatLibFi / Finto-suggestions

Finto-suggestions
7 stars 1 forks source link

Tageja tulee voida lisata listana #330

Open miguelvaara opened 4 years ago

miguelvaara commented 4 years ago

Tällä hetkellä API:n endointin "post /suggestions/{suggestion_id}/tags" kautta voi ehdotukselle lisätä vain yhden tagin kerrallaan. Jos rajapintaa hyödyntävän sovelluksen kautta halutaan lisätä ehdotukselle useampi tagi kerrallaan, joudutaan jokaisen tagin kohdalla kutsumaan rajapintaa uudelleen, mikä aiheuttaa aivan tarpeetonta kuormaa verkolle ja käytön hitautta. On täysin mahdollinen skenaario, että yhdelle ehdotukselle lisätään useampi tagi "kerralla"

osma commented 4 years ago

En vastusta, mutta epäilen kyllä että vaikutus suorituskykyyn on aika minimaalinen. En ehkä hirveästi tähän paukkuja laittaisi.

miguelvaara commented 4 years ago

@osma Laitoin Milestone 1 -tagin, mutta en itsekään näe tätä kriittisimpänä asiana.

Suorituskyky tulee siinä kohtaa yhdeksi osatekijäksi muiden joukossa, jos muutenkin on paljon samanaikaisia kyselyitä ja pyyntöjä frontin ja back-endin välillä usealla samanaikaisella käyttäjällä. Suorituskyky (tai heikot vasteajat) on ollut ongelma ehdotusjärjestelmän kanssa ja siksi tietty katseen suunta siihenkin asiaan on ehkä tarpeen, mutta yksistään tämä ei varmaankaan ole kovin suorituskykyä haittaava - siltä osin olen samoilla linjoilla.

En ajatellut keskittyä tähän issueen nyt ollenkaan, lähinnä kirjasin asian ylös. On kuitenkin hassua, että endpoint on rakennettu vain yhdelle tagin lisäämiselle eikä tag-listalle :thinking: