search

jorgejcabrera / rent-app-service

0 stars 0 forks source link

no pagina y filtra en memoria #5

Open maurociancio opened 2 years ago

maurociancio commented 2 years ago

puede ser peligroso https://github.com/jorgejcabrera/rent-app-service/blob/cb8302b1891bcd4b23c6381ca89f201da98f504a/src/main/groovy/com/fiuba/rent_app/domain/item/service/ItemServiceAdapter.groovy#L37

jorgejcabrera commented 1 year ago

por que?

maurociancio commented 1 year ago

estas levantando en memoria todos los items y luego quedandote con los que no estan siendo usados. eso instancia todos los objetos y luego tiras lo que no matchean el filtro. ademas, los items pueden ser muchos, quizas haya que mostrarlos paginados para que no requiera una cantidad zarpada de heap, con el potencial out of memory error. esto para el tp va, imagino que igual ya lo tenes presente en la vida real.

jorgejcabrera commented 1 year ago

aah, ahí va. Si tal cual, esto en una base de datos real rompe, claramente pero para esta poc me resulta útil. De hecho para no ahcer esto debería hacer un endpoint search que reciba por parámetro los filtros de búsqueda, o bien que la respuesta sea páginado. Ahí entendí.