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
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 |