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