1. The docs should be clarified a little. For instance, it should have a
link back to the definition of a prefix search (12.3.2). I included my
doc suggestions as an attachment.
Thank you, merged
2. dsynonym_init() uses findwrd() in a slightly confusing (and perhaps
fragile) way. After calling
2009/8/6 Teodor Sigaev teo...@sigaev.ru:
1. The docs should be clarified a little. For instance, it should have a
link back to the definition of a prefix search (12.3.2). I included my
doc suggestions as an attachment.
Thank you, merged
2. dsynonym_init() uses findwrd() in a slightly
On Thu, 2009-08-06 at 12:19 -0400, Robert Haas wrote:
Based on these comments, do you want to go ahead and mark this Ready
for Committer?
Done, thanks Teodor.
However, on the commitfest page, the patches got updated in the wrong
places: prefix support and filtering dictionary support are
On Thu, Aug 6, 2009 at 12:53 PM, Jeff Davispg...@j-davis.com wrote:
On Thu, 2009-08-06 at 12:19 -0400, Robert Haas wrote:
Based on these comments, do you want to go ahead and mark this Ready
for Committer?
Done, thanks Teodor.
However, on the commitfest page, the patches got updated in the
On Sun, Aug 2, 2009 at 3:05 PM, Jeff Davispg...@j-davis.com wrote:
The patch looks good.
Comments:
1. The docs should be clarified a little. For instance, it should have a
link back to the definition of a prefix search (12.3.2). I included my
doc suggestions as an attachment.
2.
On Wed, 2009-08-05 at 12:34 -0400, Robert Haas wrote:
Oleg,
Are you planning to update this patch this week? If not I will set it
to Returned with Feedback.
My only comments were related to docs and comments, and I supplied a
patch as a suggested fix for the docs. Also, the patch is very
Hi,
The patch looks good.
Comments:
1. The docs should be clarified a little. For instance, it should have a
link back to the definition of a prefix search (12.3.2). I included my
doc suggestions as an attachment.
2. dsynonym_init() uses findwrd() in a slightly confusing (and perhaps
fragile)
Hi there,
attached is our patch for CVS HEAD, which adds prefix support for synonym
dictionary.
Quick example:
cat $SHAREDIR/tsearch_data/synonym_sample.syn
postgrespgsql
postgresql pgsql
postgre pgsql
gogle googl
indices index*
=# create text search dictionary syn(