Re: Phrase search vs. multi-lexeme tokens

From: Alexander Korotkov <aekorotkov(at)gmail(dot)com>
To: Neil Chen <carpenter(dot)nail(dot)cz(at)gmail(dot)com>
Cc: PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Phrase search vs. multi-lexeme tokens
Date: 2021-01-25 15:24:52
Message-ID: CAPpHfdtBgd_sCmcqKTmg4kV2DB=CoKbW5Rqdp3teXFZP4-00JQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi, Neil!

On Mon, Jan 25, 2021 at 11:45 AM Neil Chen <carpenter(dot)nail(dot)cz(at)gmail(dot)com> wrote:
>
> The following review has been posted through the commitfest application:
> make installcheck-world: tested, passed
> Implements feature: tested, passed
> Spec compliant: not tested
> Documentation: not tested
>
> Greetings,
>
> Although I am not an expert in this field, I carefully read the full-text search section in the document. I think the change is surprising, but yes, it is correct.
> I found that your patch didn't modify the regress/excepted/tsearch.out. So I updated it and carried out the regression test. It passed. Also, I manually executed some test cases, all of which were OK.

Thank you for looking into this. Yes, I've adjusted tsearch.sql
regression tests to provide reasonable exercises for the new logic,
but forgot to add tsearch.out to the patch.

BTW, you mentioned you read the documentation. Do you think it needs
to be adjusted accordingly to the patch?

------
Regards,
Alexander Korotkov

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Alexander Korotkov 2021-01-25 15:31:01 Re: Jsonpath ** vs lax mode
Previous Message Jonah H. Harris 2021-01-25 15:18:45 Re: Extensibility of the PostgreSQL wire protocol