From: | Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com> |
---|---|
To: | Alexander Korotkov <aekorotkov(at)gmail(dot)com> |
Cc: | Tomas Vondra <tv(at)fuzzy(dot)cz>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: WIP: index support for regexp search |
Date: | 2012-11-20 20:51:08 |
Message-ID: | CAFj8pRBB0U26fK5iuXSZkOrGR3SEWq3iJRjR7zmXHTNKHR5fnA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
hello
do you plan to support GiST?
Regards
Pavel
2012/11/20 Alexander Korotkov <aekorotkov(at)gmail(dot)com>:
> On Tue, Nov 20, 2012 at 3:02 AM, Tomas Vondra <tv(at)fuzzy(dot)cz> wrote:
>>
>> 2) It's common to use upper-case names for macros, but trgm.h defines
>> macro "iswordchr" - I see it's moved from trgm_op.c but maybe we
>> could make it a bit more correct?
>>
>> 3) I see there are two '#ifdef KEEPONLYALNUM" blocks right next to each
>> other in trgm.h - maybe it'd be a good idea to join them?
>>
>> 4) The two new method prototypes at the end of trgm.h use different
>> indendation than the rest (spaces only instead of tabs).
>
> These issues are fixed in attached patch.
> Additionally 3 new macros are introduced: MAX_RESULT_STATES,
> MAX_RESULT_ARCS, MAX_RESULT_PATHS. They are limiting resources usage during
> regex processing.
>
> ------
> With best regards,
> Alexander Korotkov.
>
>
> --
> Sent via pgsql-hackers mailing list (pgsql-hackers(at)postgresql(dot)org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgsql-hackers
>
From | Date | Subject | |
---|---|---|---|
Next Message | Andres Freund | 2012-11-20 21:15:33 | Re: foreign key locks |
Previous Message | Pavel Stehule | 2012-11-20 20:47:56 | review: plpgsql return a row-expression |