Re: Supporting = operator in gin/gist_trgm_ops
Tom Lane <tgl@sss.pgh.pa.us>
From: Tom Lane <tgl@sss.pgh.pa.us>
To: Julien Rouhaud <rjuju123@gmail.com>
Cc: PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Date: 2020-10-26T04:19:48Z
Lists: pgsql-hackers
Julien Rouhaud <rjuju123@gmail.com> writes: > On Mon, Oct 26, 2020 at 5:03 AM Tom Lane <tgl@sss.pgh.pa.us> wrote: >> I think you may be overoptimistic about being able to use the identical >> code path without regard for LIKE wildcards; but certainly it should be >> possible to do this with not a lot of new code. +1. > Well, that's what I was thinking too, but I tried all the possible > wildcard combinations I could think of and I couldn't find any case > yielding wrong results. As far as I can see the index scans return at > least all the required rows, and all extraneous rows are correctly > removed either by heap recheck or index recheck. But "does it get the right answers" isn't the only figure of merit. If the index scan visits far more rows than necessary, that's bad. Maybe it's OK given that we only make trigrams from alphanumerics, but I'm not quite sure. regards, tom lane
Commits
-
Reword 'simple comparison' => 'inequality' in pgtrgm.sgml
- 7adb8feb0f69 14.0 landed
-
Add missing 'the' to pgtrgm.sgml
- b7edb5d96e7c 14.0 landed
-
Handle equality operator in contrib/pg_trgm
- 935f66665022 14.0 landed