Sorry for spamming the list.
Assuming we jump over the LGPL hurdle....
Either my brain (brian?) is fried or the code at
http://www.iq-computing.de/lucene/lucene-highlight.tar.gz doesn't
really contain an implementation of TermHighlighter interface. Is that
a mistake or has that purposely been left out? My impression was that
the code includes Maik's implementation of that.
Also, I'd make a minor change to this:
public static final String highlightTerms(String text,
TermHighlighter highlighter, Query query,
Analyzer analyzer) throws IOException
{
StringBuffer newText = new StringBuffer();
I would do something like:
StringBuffer newText = new StringBuffer(text.length); // check for
text == null before?
There is also a finally block in that method that catches Exception,
but then ignores it. Perhaps that should be modified so that it
doesn't just ignore the Exception, but rather deals with it as the rest
of Lucene code deals with Exceptions...
Otis (Ich liebe Freitagen) Gospodnetic
--- [EMAIL PROTECTED] wrote:
> Hi,
>
> I just added the Highlight terms functionality by Maik Schreiber to
> the
> contributions section.
> This code essentially gets all the terms for a given query, and then
> provides an interface to highlight the text.
>
> In order to provide this functionality he had to make some changes to
>
> the core Lucene code.
>
> I am suggesting providing the a query.getTerms() method. After review
>
> and Maik approval, we might even want to use his code.
>
> I would suggest making these changes after 1.2 is released.
>
> To check out his code and PDF describing the changes, go to
> http://www.iq-computing.de/lucene/highlight.htm
>
> Please provide feedback.
>
> Thanks
>
> --Peter
>
> ----LIST OF SUGGESTED CHANGES----
> He documented it very well. Here are the list of changes.
>
> org.apache.lucene.search.BooleanQuery - add the following method:
> public BooleanClause[] getClauses()
> {
> return (BooleanClause[]) clauses.toArray(new BooleanClause[0]);
> }
>
> org.apache.lucene.search.MultiTermQuery - mark getQuery() public
>
> org.apache.lucene.search.PhraseQuery - add the following method:
> public Term[] getTerms()
> {
> return (Term[]) terms.toArray(new Term[0]);
> }
>
> org.apache.lucene.search.PrefixQuery - mark getQuery() public
>
> org.apache.lucene.search.RangeQuery - mark getQuery() public
>
> org.apache.lucene.search.TermQuery - add the following method:
> public Term getTerm()
> {
> return term;
> }
>
__________________________________________________
Do You Yahoo!?
Got something to say? Say it better with Yahoo! Video Mail
http://mail.yahoo.com
--
To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
