← Back to team overview

zeitgeist team mailing list archive

Thread
Date

Re: lp:~thekorn/zeitgeist/fix-641100-min-max-use-index into lp:zeitgeist

To: Markus Korn <thekorn@xxxxxx>
From: Mikkel Kamstrup Erlandsen <mikkel.kamstrup@xxxxxxxxx>
Date: Fri, 17 Sep 2010 13:22:43 -0000
In-reply-to: <20100917083059.19584.20500.launchpad@loganberry.canonical.com>
Reply-to: mp+35786@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

review approve

Looks good, i'm just wondering how it fares with an empty db? But I
guess the unit tests will catch this if it's an issue.
-- 
https://code.launchpad.net/~thekorn/zeitgeist/fix-641100-min-max-use-index/+merge/35786
Your team Zeitgeist Framework Team is requested to review the proposed merge of lp:~thekorn/zeitgeist/fix-641100-min-max-use-index into lp:zeitgeist.

References

[Merge] lp:~thekorn/zeitgeist/fix-641100-min-max-use-index into lp:zeitgeist
From: Markus Korn, 2010-09-17