Thanks for the revert, I completely agree with both of your explanations.
Thanks
Anudeep Samaiya
> On 02-Jan-2020, at 21:51, charettes wrote:
>
> Hey there!
>
> The current code assumes that .get() will likely match one result which
> should be the case
> most of the time and limit the number
Hey there!
The current code assumes that .get() will likely match one result which
should be the case
most of the time and limit the number of possible results to prevent
catastrophic matches
Your patch has the side effect of performing an additional COUNT query for
every single
QuerySet.get()
Hi Anudeep
Your change makes get() perform an extra query for count() before it . This
would be a regression for most uses of get().
get() is not intended for use "when multiple objects exists in very big
numbers". You may want to perform a single query, something like
Model.objects.filter(id__in