Re: Commitfest: patches Ready for Committer

From: Michael Paquier <michael(dot)paquier(at)gmail(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Heikki Linnakangas <hlinnakangas(at)vmware(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Commitfest: patches Ready for Committer
Date: 2014-10-06 23:47:53
Message-ID: CAB7nPqRhFW1PjjWzH+4Xtnop1vdUymEV=zS45uho6gxVVxnvJw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Mon, Oct 6, 2014 at 10:53 PM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:

> The levenshtein-distance thingy seems to still be a topic of debate
> as well, both as to how we're going to refactor the code and as to
> what the exact hinting rules ought to be. If some committer wants
> to take charge of it and resolve those issues, fine; but I don't want
> to see it done by just blindly committing whatever the last submitted
> version was.
>

My 2c. I imagine that in this case the consensus is going to be first:
- Move a minimum of the core functions of fuzzystrmatch and rename them
(str_distance?)
- Do not dump fuzzystrmatch and have the levenshtein call those functions
In any case levenshtein code needs a careful refactoring and some
additional thoughts first before the hint code is touched.
Regards.
--
Michael

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Michael Paquier 2014-10-06 23:50:23 Re: Failure with make check-world for pgtypeslib/dt_test2 with HEAD on OSX
Previous Message Jim Nasby 2014-10-06 23:41:11 Re: Add regression tests for autocommit-off mode for psql and fix some omissions