| From: | Oleg Bartunov <obartunov(at)gmail(dot)com> |
|---|---|
| To: | Marcelo Zabani <mzabani(at)gmail(dot)com> |
| Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Pgsql Hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: Html parsing and inline elements |
| Date: | 2016-04-30 21:43:37 |
| Message-ID: | CAF4Au4zH2Rc4=-nj92y=NVcF_QTN1PCHE0Rj8_mh9zA0NH9JoA@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Wed, Apr 13, 2016 at 6:57 PM, Marcelo Zabani <mzabani(at)gmail(dot)com> wrote:
> Hi, Tom,
>
> You're right, I don't think one can argue that the default parser should
> know HTML.
> How about your suggestion of there being an HTML parser, is it feasible? I
> ask this because I think that a lot of people store HTML documents these
> days, and although there probably aren't lots of HTML with words written
> along multiple inline elements, it would certainly be nice to have a proper
> parser for these use cases.
>
> What do you think?
>
I think it could be useful separate parser. But the problem is how to fully
utilize it to facilitate ranking, for example, words in title could be
considered more important than in the body, etc. Currently, setweight()
functions provides this separately from parser.
Parser outputs tokid and token:
select * from ts_parse('default','<title>parser</title><body>text</body>');
tokid | token
-------+----------
13 | <title>
1 | parser
13 | </title>
13 | <body>
1 | text
13 | </body>
(6 rows)
If we change parser to output also rank flag, then we could use it to
assign different weights.
>
> On Wed, Apr 13, 2016 at 11:09 AM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>
>> Marcelo Zabani <mzabani(at)gmail(dot)com> writes:
>> > I was here wondering whether HTML parsing should separate tokens that
>> are
>> > not separated by spaces in the original text, but are separated by an
>> > inline element. Let me show you an example:
>>
>> > *SELECT to_tsvector('english', 'Hello<p>neighbor</p>, you are
>> > <strong>n</strong>i<em>ce</em>')*
>> > *Results:** "'ce':7 'hello':1 'n':5 'neighbor':2"*
>>
>> > "Hello" and "neighbor" should really be separated, because *<p>* is a
>> block
>> > element, but "nice" should be a single word there, since there is no
>> visual
>> > separation when rendered (*<em>* and *<strong>* are inline elements).
>>
>> I can't imagine that we want to_tsvector to know that much about HTML.
>> It doesn't, really, even have license to assume that its input *is*
>> HTML. So even if you see things that look like <foo> and </foo> in the
>> string, it could easily be XML or SGML or some other SGML-like markup
>> format with different semantics for the markup keywords.
>>
>> Perhaps it'd be sane to do something like this as long as the
>> HTML-specific behavior was broken out into a separate function.
>> (Or maybe it could be done within to_tsvector as a separate parser
>> or separate dictionary?) But I don't think it should be part of
>> the default behavior.
>>
>> regards, tom lane
>>
>
>
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Bruce Momjian | 2016-04-30 21:55:08 | Re: Is pg_control file crashsafe? |
| Previous Message | Oleg Bartunov | 2016-04-30 21:07:11 | Re: Lets (not) break all the things. Was: [pgsql-advocacy] 9.6 -> 10.0 |