markharw00d wrote:
Is there any particular reason why SpanQuery introduced
  public Collection getTerms()
when we have:
   public void extractTerms(Set terms)
in Query?

I can think of no compelling reason.

I am changing the highlighter to make use of extractTerms and can either add this to SpanQuery:

 public void extractTerms(Set terms) {
     terms.addAll(getTerms());
 }

or (perhaps more riskily) consider removing getTerms() completely from SpanQueries and replace with extractTerms.

I think the latter approach is better. We'd should deprecate getTerms(), in case there is other code that calls it.

Doug

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to