I made few additional changes in RAMDirectory.cpp and MockRAMDirectory.cpp (see 
intensive_testing branch for details)
Now TestRAMDirectory works fine with MockRAMDirectory.

Veit, thanks again for your support and comments

Borek

> -----Original Message-----
> From: Kostka Bořivoj [mailto:kos...@tovek.cz]
> Sent: Saturday, August 14, 2010 11:47 PM
> To: clucene-developers@lists.sourceforge.net
> Subject: Re: [CLucene-dev] RAMDirectory testing (using MockRAMDirectory)
> 
> 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

------------------------------------------------------------------------------
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