zeitgeist team mailing list archive
-
zeitgeist team
-
Mailing list archive
-
Message #01806
Re: [Merge] lp:~thekorn/zeitgeist/fix-641100-min-max-use-index into lp:zeitgeist
Mikkel, thanks for doing the review,
yes, the case of an empty db is covered by our unittests and everything works just fine.
(
MIN/MAX always return sth. even if the table it's operating on is empty
In [7]: con = sqlite3.connect(":memory:")
In [8]: con.execute("CREATE TABLE boo(x)")
Out[8]: <sqlite3.Cursor object at 0x117da28>
In [9]: con.execute("SELECT MIN(x) FROM boo").fetchone()
Out[9]: (None,)
)
--
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