Re: [GENERAL] Fragments in tsearch2 headline

Поиск
Список
Период
Сортировка
От Sushant Sinha
Тема Re: [GENERAL] Fragments in tsearch2 headline
Дата
Msg-id 1216180441.10591.1.camel@dragflick
обсуждение исходный текст
Ответ на Re: [GENERAL] Fragments in tsearch2 headline  (Teodor Sigaev <teodor@sigaev.ru>)
Список pgsql-hackers
attached are two patches:

1. documentation
2. regression tests
 for headline with fragments.

-Sushant.

On Tue, 2008-07-15 at 13:29 +0400, Teodor Sigaev wrote:
> > Attached a new patch that:
> >
> > 1. fixes previous bug
> > 2. better handles the case when cover size is greater than the MaxWords.
>
> Looks good, I'll make some tests with  real-world application.
>
> > I have not yet added the regression tests. The regression test suite
> > seemed to be only ensuring that the function works. How many tests
> > should I be adding? Is there any other place that I need to add
> > different test cases for the function?
>
> Just add 3-5 selects to src/test/regress/sql/tsearch.sql with checking basic
> functionality and corner cases like
>   - there is no covers in text
>   - Cover(s) is too big
>   - and so on
>
> Add some words in documentation too, pls.
>
>

Вложения

В списке pgsql-hackers по дате отправления:

Предыдущее
От: Decibel!
Дата:
Сообщение: Re: Lookup penalty for VARIADIC patch
Следующее
От: "David E. Wheeler"
Дата:
Сообщение: Re: PATCH: CITEXT 2.0 v3