django-oscar / django-oscar-elasticsearch

Search app for oscar using elasticsearch
BSD 3-Clause "New" or "Revised" License
20 stars 18 forks source link

Using django's iterator is not really neccesary and add .distinct() on product queryset #62

Closed joeyjurjens closed 1 month ago

joeyjurjens commented 1 month ago

I thought that when slicing, it still consumed a lot of memory and that's why I added the use of .iterator() in the chunked util. However, after testing, slicing works fine and and it's probably better to keep it simple and use slicing.