MoreLikeThis API changes?

2007-05-30 Thread Ryan McKinley
I'm trying to build a custom MoreLikeThis implementation that will run within solr and I've run into a few API hurdles... 1. Can MLT.java be modified to optionally take the Similarity implementation in the constructor? Currently it is hardcoded to: private Similarity similarity = new

Re: MoreLikeThis API changes?

2007-05-30 Thread Grant Ingersoll
On May 30, 2007, at 2:45 AM, Ryan McKinley wrote: I'm trying to build a custom MoreLikeThis implementation that will run within solr and I've run into a few API hurdles... 1. Can MLT.java be modified to optionally take the Similarity implementation in the constructor? Currently it is

Re: MoreLikeThis API changes?

2007-05-30 Thread Ryan McKinley
2. Do retrieveTerms(int docNum) and createQuery(PriorityQueue q) need to be private? Can they be public? If not public, could they at least be protected? I would think protected would be fine, what is your case for it being public? From the solr RequestHandler, I want to return the

Re: MoreLikeThis API changes?

2007-05-30 Thread mark harwood
, 2007 5:16:16 PM Subject: Re: MoreLikeThis API changes? 2. Do retrieveTerms(int docNum) and createQuery(PriorityQueue q) need to be private? Can they be public? If not public, could they at least be protected? I would think protected would be fine, what is your case for it being

Re: MoreLikeThis API changes?

2007-05-30 Thread Ryan McKinley
mark harwood wrote: I want to return the interesting terms used for MLT Could you do this using Query.extractTerms() on the rewritten version of the MoreLikeThis query (a BooleanQuery)? thanks! that works and avoids the PriorityQueue traverstal problems. I can even get the boost