lucene-solr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Anders Melchiorsen (JIRA)" <j...@apache.org>
Subject [jira] Commented: (SOLR-1394) HTML stripper is splitting tokens
Date Fri, 16 Oct 2009 22:19:31 GMT

    [ https://issues.apache.org/jira/browse/SOLR-1394?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12766735#action_12766735
] 

Anders Melchiorsen commented on SOLR-1394:
------------------------------------------

Thanks, that sounds great.

There is an existing off-by-one error in the numWhitespace calculation with hexadecimal numeric
entities.

I noticed that while reworking the patch, but did not bother to report it in here because
I was annoyed from being ignored. Now you got me in a better mood, so I can fix that error
if you like?


> HTML stripper is splitting tokens
> ---------------------------------
>
>                 Key: SOLR-1394
>                 URL: https://issues.apache.org/jira/browse/SOLR-1394
>             Project: Solr
>          Issue Type: Bug
>          Components: Analysis
>    Affects Versions: 1.4
>            Reporter: Anders Melchiorsen
>         Attachments: SOLR-1394.patch, SOLR-1394.patch
>
>
> The Solr HTML stripper is replacing any removed HTML with whitespace. This is to keep
offsets correct for highlighting.
> However, as was already pointed out in SOLR-42, this means that any token containing
an HTML entity will be split into several tokens. That makes the HTML stripper completely
unreliable for international text (and any text is potentially interantional).
> The current code is actually deficient for BOTH highlighting and indexing, where the
previous incarnation (that did not insert spaces) only had problems with highlighting.
> The only workaround is to not use entities at all, which is impossible in some situations
and inconvenient in most situations. If the client is required to transform entities before
handing it to Solr, it might as well be required to also strip tags, and then the HTML stripper
would not be needed at all.
> Today, we have a better solution that can be used: offset correction. We can then avoid
inserting extra whitespace, but still get correct offsets. The attached patch implements just
that.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message