Re: WIP: index support for regexp search - Mailing list pgsql-hackers

From Pavel Stehule
Subject Re: WIP: index support for regexp search
Date
Msg-id CAFj8pRBB0U26fK5iuXSZkOrGR3SEWq3iJRjR7zmXHTNKHR5fnA@mail.gmail.com
Whole thread Raw
In response to Re: WIP: index support for regexp search  (Alexander Korotkov <aekorotkov@gmail.com>)
Responses Re: WIP: index support for regexp search  (Alexander Korotkov <aekorotkov@gmail.com>)
List pgsql-hackers
hello

do you plan to support GiST?

Regards

Pavel

2012/11/20 Alexander Korotkov <aekorotkov@gmail.com>:
> On Tue, Nov 20, 2012 at 3:02 AM, Tomas Vondra <tv@fuzzy.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@postgresql.org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgsql-hackers
>



pgsql-hackers by date:

Previous
From: Pavel Stehule
Date:
Subject: review: plpgsql return a row-expression
Next
From: Andres Freund
Date:
Subject: Re: foreign key locks