[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: [dak/master] Use distinct() to avoid listing duplicated claimed overrides



2013/4/26 Ansgar Burchardt <ansgar@debian.org>:
> On 04/26/2013 12:56, Luca Falavigna wrote:
>> +    else:
>> +        return None
>
> Why not just return an empty list instead of None?

Just to provide the check below...

>> +    if claimed and claimed.count():
>
> I think this does two SQL queries: one to check count() > 0 and another
> one to iterate over the packages.

... but the solution you suggested is way better. Fixed, thanks!


Reply to: