I tested once more our code in order to check the memory leaks.
Here is the list of leaking test suites:

- testatomicupdates
- testindexreader
- testanalyzers
- testRAMDirectory
- testindexwriter
- testIndexModifier
- testAddIndexesNoOptimize
- testsearch
- testRangeFilter
- testConstantScoreQueries
- testsort
- testdatefilter
- testutf8
- testBitSet
- testExtractTerms

Something must have changed in the code during the merges because I am sure
that at least the testExtractTerms test didn't produce memory leaks.

Let us get rid of the leaks! I will start from the bottom 
with the testExtractTerms test. Should we create a new branch for this issue?

Jiri

        

-----Original Message-----
From: Veit Jahns [mailto:nuncupa...@googlemail.com] 
Sent: Sunday, September 19, 2010 9:51 PM
To: clucene-developers@lists.sourceforge.net
Subject: Re: [CLucene-dev] Current branches state

2010/9/16 Itamar Syn-Hershko <ita...@code972.com>:
> Veit, Borek and Jiri - please have a look at master and make sure the
> merge did not ruin your changes (this sometimes happens), and that I
> resolved all merge conflicts correctly. Once you confirmed the merge,
> please test the branch as intensively as possible. After you have done
> this, please remove your merged branches from the repo.

Compiles and runs on FreeBSD as well as Win7/VS2010. But the one of
the tests for my StringBuffer extension failed. Fixed it. I needed
this extension for the GermanAnalyzer. But I havn't completed the port
yet, so the merge of veit_working into master was may be a little bit
early. But as this is a contrib no harm will be done to the core.

>  From now on, everyone with a repo access bit, please have a branch per
> feature / fix and make sure it doesn't become stale until it is accepted
> and merged to master. I'll do my best to merge features and fixes as
> quickly as possible - having one branch per change is going to make this
> simpler anyway.

But in the case of the failing StringBuffer tests mentioned above I
pushed the commits directly to the master branch (sorry). The remaing
ported code for the GermanAnalyzer will be pushed to a separate
branch.

Borek/Jiří, one of you mentioned, that there observed memory leaks in
tests. Are they still there? I ask, because veit_working had many
memory leak fixes.

Veit

------------------------------------------------------------------------------
Start uncovering the many advantages of virtual appliances
and start using them to simplify application deployment and
accelerate your shift to cloud computing.
http://p.sf.net/sfu/novell-sfdev2dev
_______________________________________________
CLucene-developers mailing list
CLucene-developers@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/clucene-developers
------------------------------------------------------------------------------
Start uncovering the many advantages of virtual appliances
and start using them to simplify application deployment and
accelerate your shift to cloud computing.
http://p.sf.net/sfu/novell-sfdev2dev
_______________________________________________
CLucene-developers mailing list
CLucene-developers@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/clucene-developers

Reply via email to