From: | Alexander Korotkov <aekorotkov(at)gmail(dot)com> |
---|---|
To: | Heikki Linnakangas <hlinnakangas(at)vmware(dot)com> |
Cc: | pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Terminology issue: suffix tree |
Date: | 2013-05-08 12:49:50 |
Message-ID: | CAPpHfdv+FiHM6HfL1oZwZeVQcuxW6J7MEnzxhL6e_DCZ63SfKg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Wed, May 8, 2013 at 3:50 PM, Heikki Linnakangas
<hlinnakangas(at)vmware(dot)com>wrote:
> On 06.05.2013 14:10, Alexander Korotkov wrote:
>
>> On Sat, May 4, 2013 at 10:27 PM, Alexander Korotkov<aekorotkov(at)gmail(dot)com>
>> **wrote:
>>
>>> In suffix tree we insert every suffix of source string into the tree.
>>>
>>> http://en.wikipedia.org/wiki/**Suffix_tree<http://en.wikipedia.org/wiki/Suffix_tree>
>>> Actually opclass implemented radix tree or patricia tree.
>>> http://en.wikipedia.org/wiki/**Radix_tree<http://en.wikipedia.org/wiki/Radix_tree>
>>> Likely we need a patch to rename it in all the places it mentioned.
>>>
>>
>> Patch is attached.
>>
>
> Thanks, committed.
Thanks!
> Apparently, we have same issue in contrib/unaccent.
>>
>
> Yeah. The data structure in contrib/unaccent seems to be a plain old trie,
> rather than a radix trie, though. According to wikipedia at least, the
> difference is that in a radix tree, the edges are labeled with sequences of
> elements, rather than single elements. Want to patch that too?
Agree, trie is most comforming term here. Patch is attached.
------
With best regards,
Alexander Korotkov.
Attachment | Content-Type | Size |
---|---|---|
unaccent-suffix-fix.patch | application/octet-stream | 5.7 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2013-05-08 12:59:54 | Re: [COMMITTERS] pgsql: Fix permission tests for views/tables proven empty by constraint |
Previous Message | Heikki Linnakangas | 2013-05-08 11:50:03 | Re: Terminology issue: suffix tree |