On Mon, Nov 16, 2020 at 2:13 AM Jeff Janes <jeff.ja...@gmail.com> wrote: > On Sat, Nov 14, 2020 at 12:31 AM Alexander Korotkov <aekorot...@gmail.com> > wrote: >> I went through and revised this patch. I made the documentation >> statement less categorical. pg_trgm gist/gin indexes might have lower >> performance of equality operator search than B-tree. So, we can't >> claim the B-tree index is always not needed. Also, simple comparison >> operators are <, <=, >, >=, and they are not supported. > > Is "simple comparison" here a well-known term of art? If I read the doc as > committed (which doesn't include the sentence above), and if I didn't already > know what it was saying, I would be left wondering which comparisons those > are. Could we just say "inequality operators"?
You're right. "Simple comparison" is vague, let's replace it with "inequality". Pushed, thanks! ------ Regards, Alexander Korotkov
