lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "David Smiley (JIRA)" <>
Subject [jira] [Reopened] (LUCENE-7559) UnifiedHighlighter: Make Passage public for extensibility
Date Wed, 11 Jan 2017 22:23:17 GMT


David Smiley reopened LUCENE-7559:

Reopening.  [~dweiss] helpfully pointed out that the {{OffsetsEnum}} class in the UnifiedHighlighter
should also have it's methods be made public, otherwise someone subclassing to override {{FieldHighlighter#highlightOffsetsEnums}}
can't really do anything useful.  And doing that is the primary motivation for making Passage

Additionally, it'd be good to add a basic test in TestUnifiedHighlighterExtensibility showing
we got this right.

> UnifiedHighlighter: Make Passage public for extensibility
> ---------------------------------------------------------
>                 Key: LUCENE-7559
>                 URL:
>             Project: Lucene - Core
>          Issue Type: Improvement
>          Components: modules/highlighter
>            Reporter: David Smiley
>            Assignee: David Smiley
>            Priority: Minor
>             Fix For: 6.4
>         Attachments: LUCENE-7559.patch
> (UnifiedHighlighter) has a variety of package-level fields and methods that
are accessed by FieldHighlighter.highlightOffsetsEnums (the kernel of the UH).  It should
be possible for people to extend this method to tweak the process of producing passages. 
That method is extensible but is too closed.  Even if one were to write their
own equivalent to, it would be impossible to then use PassageScorer or PassageFormatter.

This message was sent by Atlassian JIRA

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message