On Tue, Aug 4, 2009 at 11:18 AM, Michael McCandless<luc...@mikemccandless.com> wrote: > On Tue, Aug 4, 2009 at 9:56 AM, Yonik Seeley<yo...@lucidimagination.com> > wrote: >> OK, it sounds like it should always be the same as SortField.getReversed() >> then? >> In which case it seems redundant to pass it into >> SortField.getComparator() since SortField already keeps track of it? > > Duh, right. I'll remove as an arg and simply use SortField's reverse > attr directly. > > In fact I think SortField.getComparator should be package private?
It would be nice to keep - Solr already uses it :-) -Yonik http://www.lucidimagination.com --------------------------------------------------------------------- To unsubscribe, e-mail: java-dev-unsubscr...@lucene.apache.org For additional commands, e-mail: java-dev-h...@lucene.apache.org
