This has been fixed and will be in the next 8.3 minor release.

---------------------------------------------------------------------------

Tom Lane wrote:
> "Denis Monsieur" <dmonsi...@gmail.com> writes:
> > The problem is a space being added to text in the form of
> > http://some.url/path
> > Compare the output:
> 
> > shs=# SELECT ts_headline('http://some.url', to_tsquery('sometext'));
> >    ts_headline
> > -----------------
> >  http://some.url
> > (1 row)
> 
> > shs=# SELECT ts_headline('http://some.url/path', to_tsquery('sometext'));
> >       ts_headline
> > -----------------------
> >  http:// some.url/path
> > (1 row)
> 
> I looked into this, and it seems that the problem is that
> generateHeadline() emits a space for any token marked as replace = 1.
> I think it probably shouldn't emit anything at all.  AFAICS the cases
> where replace will get set are token types URL, TAG, NUMHWORD,
> ASCIIHWORD, HWORD.  For URL and the HWORD variants the space is
> certainly undesirable, because these token types are just respecifying
> text that is also covered by their component tokens.  The only case
> where you could make an argument that the space is useful is TAG,
> as in
> 
> regression=# SELECT ts_headline('http<foo>blah', to_tsquery('sometext'));
>  ts_headline 
> -------------
>  http blah
> (1 row)
> 
> But it seems to me to be at least as plausible that you should get
> nothing as that you should get a space for a removed tag.
> 
> Comments?
> 
>                       regards, tom lane
> 
> -- 
> Sent via pgsql-bugs mailing list (pgsql-bugs@postgresql.org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgsql-bugs

-- 
  Bruce Momjian  <br...@momjian.us>        http://momjian.us
  EnterpriseDB                             http://enterprisedb.com

  + If your life is a hard drive, Christ can be your backup. +

-- 
Sent via pgsql-bugs mailing list (pgsql-bugs@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-bugs

Reply via email to