On Mon, Mar 26, 2018 at 9:51 PM, Dmitry Ivanov <d.ivanov@postgrespro.ru> wrote:
>> Recently I worked with the old version of the patch and found a bug.
>> So, I think it is better to notify you immediately, so you can fix it in
>> rebased/revised version.
>>
>> I noticed, that operator AROUND(N) works only
>> in case of non-negative operands. If any of the operands is negative, it
>> behaves as phrase operator <N>. It is caused by lack of TS_NOT_EXAC
>> flag and AROUND(N) operator check in function TS_phrase_execute in
>> branches for negated operands.
>
> Good to know, thanks! To be honest, I' sure that Theodor is right: it's
> better to implement AROUND(N) operator using <N, M> when it's committed. The
> following version of patch won't support AROUND(N). I have to fix a few more
> questionable things, though.
Hi,
I took a quick look at the language in the last version of the patches.
Patch 01:
+ errmsg("Invalid AROUND(X) operator!")));
s/I/i/;s/!//
+ errmsg("Missing ')' in AROUND(X) operator")));
s/M/m/
Patch 03 (the documentation) needed some proof-reading. I've attached
a new version of that patch with some small suggested improvements.
Questions I had while reading the documentation without looking at the
code: Is there anything to_tsquery() can do that
websearch_to_tsquery() can't? The documentation doesn't mention
parentheses, but I can see that they are in fact supported from the
regression test. Would it be OK to use user-supplied websearch
strings? Ie can it produce a syntax error? Well clearly it can, see
above -- perhaps that should be explicitly documented? Is there any
way to write OR as a term (that's a valuable non-stopword in French)?
It seems like AROUND(x) should be documented also more generally for
tsquery, but I see there is some discussion about how that should
look.
By the way, not this patch's fault, but I noticed that commit
f5f1355dc4d did this:
- (errmsg("query contains only
stopword(s) or doesn't contain lexeme(s), ignored")));
+ (errmsg("text-search query contains
only stop words or doesn't contain lexemes, ignored")));
But the old test still appears in an example in doc/src/sgml/textsearch.sgml.
--
Thomas Munro
http://www.enterprisedb.com