Re: [HACKERS] [sqlsmith] Missing CHECK_FOR_INTERRUPTS in tsquery_rewrite

2016-10-30 Thread Tom Lane
I wrote: > Also, I think this is outright *wrong* for phrase search --- dropping some > of the child nodes without any other adjustment isn't valid is it? After further digging, it seems there's no bug because the tree is originally binary and QTNTernary won't try to flatten OP_PHRASE nodes. So

Re: [HACKERS] [sqlsmith] Missing CHECK_FOR_INTERRUPTS in tsquery_rewrite

2016-10-30 Thread Tom Lane
Andreas Seltenreich writes: > testing with sqlsmith yielded an uncancellable backend hogging CPU time. > Gdb showed it was busy in findeq() of tsquery_rewrite.c. This function > appears to have exponential complexity wrt. the size of the involved > tsqueries. The following