Hi Veit,

Good catch, it works fine now, thanks.

Borek

> -----Original Message-----
> From: Kostka Bořivoj [mailto:kos...@tovek.cz]
> Sent: Thursday, August 12, 2010 10:31 AM
> To: clucene-developers@lists.sourceforge.net
> Subject: Re: [CLucene-dev] RAMDirectory testing (using MockRAMDirectory)
> 
> Thanks, I'll apply and test
> 
> Borek
> 
> > -----Original Message-----
> > From: Veit Jahns [mailto:nuncupa...@googlemail.com]
> > Sent: Thursday, August 12, 2010 8:56 AM
> > To: clucene-developers@lists.sourceforge.net
> > Subject: Re: [CLucene-dev] RAMDirectory testing (using MockRAMDirectory)
> >
> > 2010/8/11 Veit Jahns <nuncupa...@googlemail.com>:
> > > 2010/8/11 Kostka Bořivoj <kos...@tovek.cz>:
> > >>> > 2)     MockRAMDirectory::close always throws an exception because
> > >>> > segments.gen is in list of open files, no idea if this is a bug in
> > >>> > RAMDirectory or MockRAMDirectory
> > >>>
> > >>> Why do you use MockRAMDirectory here? I ported this class from
> > >>> JLucene, because I needed once a directory implementation that
> > >>> intentionally throws exceptions. And because the close-methods of
> > >>> RAMDirectory and MockRAMDirectory are identical to the JLucene
> > >>> methods, I'd say that's neither a bug of RAMDirectory nor 
> > >>> MockRAMDirectory.
> > >>
> > >> I use MockRAMDirectory just because it is used in original java test
> > >> I ported. It is used in many tests in test/index folder so I wanted
> > >> to test MockRAMDirectory itself before using it in other tests.
> > >
> > > You are right. Something is odd there. I will take a close look tomorrow.
> >
> > The Bug seems to be in SegmentInfos.cpp. In some case the open generation 
> > file
> will
> > not be closed (diff attached).
> >
> > Veit
> 
> ------------------------------------------------------------------------------
> This SF.net email is sponsored by
> 
> Make an app they can't live without
> Enter the BlackBerry Developer Challenge
> http://p.sf.net/sfu/RIM-dev2dev
> _______________________________________________
> CLucene-developers mailing list
> CLucene-developers@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/clucene-developers

------------------------------------------------------------------------------
This SF.net email is sponsored by 

Make an app they can't live without
Enter the BlackBerry Developer Challenge
http://p.sf.net/sfu/RIM-dev2dev 
_______________________________________________
CLucene-developers mailing list
CLucene-developers@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/clucene-developers

Reply via email to