Code4PuertoRico / suministrospr

Suministros Puerto Rico - Entérate, Ayuda, Informa
https://suministrospr.com
Apache License 2.0
42 stars 17 forks source link

Optimize Suministro “group by” #84

Closed arielo closed 4 years ago

arielo commented 4 years ago

The way suministró records are fetched could be problematic as the amount of records grow. The group by could be done at a DB query level reducing operations and memory usage.

https://github.com/Code4PuertoRico/suministrospr/blob/a0f6a70cc2f8894dd79a41a43e67f35d851e51ea/suministrospr/suministros/views.py#L26

@gcollazo This is not a blocker for v1 just something to keep an eye out for.

github-actions[bot] commented 4 years ago

Gracias por por tu aportación! El equipo le dara un vistazo pronto.

rnegron commented 4 years ago

I can take this task.

jpadilla commented 4 years ago

Gracias @arielo @rnegron 🎉