zeitgeist team mailing list archive
-
zeitgeist team
-
Mailing list archive
-
Message #04772
Re: [Merge] lp:~zeitgeist/zeitgeist/fts++ into lp:zeitgeist
Review: Approve
OK, merging it, but there's some outstanding stuff:
* Important: the TableLookup in FTS can currently explode. The new schema version needs to add AUTOINCREMENT to the `id' row of all tables in TableLookup. We should do this before releasing a new tarball.
* TableLookup.get_value: please prepare the query in the constructor.
* Configure isn't checking for xapian being there
* Add a flag to disable FTS? (keeping the Xapian dependency avoidable)
--
https://code.launchpad.net/~zeitgeist/zeitgeist/fts++/+merge/92022
Your team Zeitgeist Framework Team is subscribed to branch lp:zeitgeist.
Follow ups
References