Bug#761867: Webapp: Add conjunctive search for ctags
[ just documenting here a quick summary of IRC discussions ]
On Thu, Mar 12, 2015 at 02:33:03PM +0100, Orestis Ioannou wrote:
> I have attached the patch for the conjunctive search.
> Raw sql commands are the following:
The patch looks good, but there are concerns that this feature will
allow to DoS the DB, given the ctags table currently contains ~380
million tuples.
We've pinged friendly neighborhood Postgres experts (Myon, mnencia) who
are looking into this to give feedback about: whether the feature is a
good idea at all and/or whether there are better ways to implement this
query.
I duly notice that we already have non-conjunctive ctags-based search.
And it's not given us problems thus far. If we can make the conjunctive
query, performance-wise, "as bad as" the non-conjunctive version, then
it's probably good to do.
Cheers.
--
Stefano Zacchiroli . . . . . . . zack@upsilon.cc . . . . o . . . o . o
Maître de conférences . . . . . http://upsilon.cc/zack . . . o . . . o o
Former Debian Project Leader . . @zack on identi.ca . . o o o . . . o .
« the first rule of tautology club is the first rule of tautology club »
Reply to: