Hi Daniel Thanks for investigating this.
> - keep the optimize logic but use forceMerge method instead > > Ok, it says "This is a horribly costly operation, especially when you pass a small maxNumSegments; usually you should only call this if the index is static (will no longer be changed)." So I'm not sure what value to set to maxNum. What do you suggest? > Subsequently I will look into CRIS and check if optimize can be replaced by > a merge policy or some other method. > Good. Having had a closer look into the current CRIS code I was wondering about the facet iimplementation why this doesn't seem to be mapped to the lucene.facet classes, i.e. it is countingthe facets manually rather than using org.apache.lucene.facet.search.CountFacetRequest, does this scale? Cheers, Reto > > > > 2013/4/19 Reto Bachmann-Gmür <[email protected]> > > > Ok good. I'm currently using the system with some data. If no problem > occur > > with the new version I'll remove those methods from the interfaces too. > > > > Cheers, > > Reto > > > > > > On Fri, Apr 19, 2013 at 2:39 PM, Daniel Spicar <[email protected]> > wrote: > > > > > Hi reto, > > > > > > No I did not check anything recently. I can review the changes if you > > like. > > > If optimize is not needed anymore that is good news. AFAIK it has been > > > mostly needed to keep the number of open file descriptors limited. > > > > > > Daniel > > > > > > > > > 2013/4/19 Reto Bachmann-Gmür <[email protected]> > > > > > > > Hi Daniel > > > > > > > > I'm wondering if you had a look at the recent changes to CRIS to work > > > with > > > > a more recent lucene version. > > > > > > > > With this update the optimize methods have become obsolete. > > > > > > > > Cheers, > > > > Reto > > > > > > > > > >
