Hi, On Thu, Jun 23, 2011 at 2:39 PM, Yonik Seeley <yo...@lucidimagination.com>wrote:
> On Wed, Jun 22, 2011 at 10:30 PM, Chris Male (JIRA) <j...@apache.org> > wrote: > > > > [ > https://issues.apache.org/jira/browse/LUCENE-3232?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel] > > > > Chris Male updated LUCENE-3232: > > ------------------------------- > > > > Description: > > Solr makes use of the MutableValue* series of classes to improve > performance of grouping by FunctionQuery (I think). As such they are used > in ValueSource implementations. Consequently we need to move these classes > in order to move the ValueSources. > > > > As Yonik pointed out, these classes have use beyond just FunctionQuerys > and might be used by both Solr and other modules. However I don't think > they belong in Lucene core, since they aren't really related to search > functionality. Therefore I think we should put them into a Common module, > which can serve as a dependency to Solr and any module. > > My point was more about thinking about the package name up front - if > it's in a more generic package, it can be easily moved to another > module later if another module needs it. Up to you if you want to > create that common module up front, or wait until it's needed of > course. > Yeah, you made a really good point. I think its best to go for the module straight away since its not a huge amount of effort and will end up being the right way in the end. Thanks for the review, most appreciated. > > -Yonik > http://www.lucidimagination.com > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org > For additional commands, e-mail: dev-h...@lucene.apache.org > > -- Chris Male | Software Developer | JTeam BV.| www.jteam.nl