Re: Fwd: pg_trgm word_similarity inconsistencies or bug

From: Alexander Korotkov <a(dot)korotkov(at)postgrespro(dot)ru>
To: Jan Przemysław Wójcik <jan(dot)przemyslaw(dot)wojcik(at)gmail(dot)com>
Cc: pgsql-bugs(at)postgresql(dot)org
Subject: Re: Fwd: pg_trgm word_similarity inconsistencies or bug
Date: 2017-11-07 16:24:27
Message-ID: CAPpHfduZEGYcj3-31ONbReCFDYah1RZKv3a3WRGV=1tN43Zrpw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs pgsql-hackers

Hi!

On Tue, Nov 7, 2017 at 3:51 PM, Jan Przemysław Wójcik <
jan(dot)przemyslaw(dot)wojcik(at)gmail(dot)com> wrote:

> Hi,
>
> my statement about the function usefulness was probably too categorical,
> though I had in mind the current name of the function.
>
> I'm afraid that creating a function that implements quite different
> algorithms depending on a global parameter seems very hacky and would lead
> to misunderstandings. I do understand the need of backward compatibility,
> but I'd opt for the lesser evil. Perhaps a good idea would be to change the
> name to 'substring_similarity()' and introduce the new function
> 'word_similarity()' later, for example in the next major version release.
>

Good point. I've no complaints about that. I'm going to propose
corresponding patch to the next commitfest.

------
Alexander Korotkov
Postgres Professional: http://www.postgrespro.com
The Russian Postgres Company

In response to

Responses

Browse pgsql-bugs by date

  From Date Subject
Next Message Tom Lane 2017-11-07 22:29:41 Re: BUG #14890: Error grouping by same column twice using FDW
Previous Message Raphael Hertzog 2017-11-07 16:23:20 Re: Fails to work on live images due to fsync() on pg_commit_ts before doing any write there

Browse pgsql-hackers by date

  From Date Subject
Next Message David Steele 2017-11-07 16:42:21 Re: Exclude pg_internal.init from base backup
Previous Message Geoff Winkless 2017-11-07 16:07:42 Re: MERGE SQL Statement for PG11