Re: Prefix support for synonym dictionary
| От | Jeff Davis |
|---|---|
| Тема | Re: Prefix support for synonym dictionary |
| Дата | |
| Msg-id | 1249239924.4765.6926.camel@jdavis обсуждение исходный текст |
| Ответ на | Prefix support for synonym dictionary (Oleg Bartunov <oleg@sai.msu.su>) |
| Ответы |
Re: Prefix support for synonym dictionary
Re: Prefix support for synonym dictionary |
| Список | pgsql-hackers |
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) way. After calling findwrd(), the "end" pointer is pointing at
either the end of the string, or the *; depending on whether the string
ends in * and whether flags is NULL. I only mention this because I had
to take a more careful look to see what was happening. Perhaps add a
comment to make it more clear?
3. The patch looks for the special byte '*'. I think that's fine,
because we depend on the files being in UTF-8 encoding, where it's the
same byte. However, I thought it was worth mentioning in case we want to
support other encodings for text search files later.
Regards,
Jeff Davis
Вложения
В списке pgsql-hackers по дате отправления: