| 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: | Whole Thread | Raw Message | 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 |