Andreas L Delmelle wrote:

On Nov 15, 2007, at 16:30, Chris Bowditch wrote:

Thanks for the diff. Unfortunately I have been unsuccessful in applying it after several attempts. First I tried using Tortoise SVN client, then I downloaded GNUWin32 Patch and that fails to apply all but hunk 7. I also asked a colleague working on Linux to try and apply the patch but it fails for him too (although one more hunk is successful)

I guess I could manually make the updates, but I would prefer to work out whats going wrong here to avoid similar problems in the future and to minimize the risk of error.

Updated diff in attach. No idea why the patching would fail on your end... maybe something to do with encoding?
Saved the file now explicitly with ISO-encoding, just to be sure.

Thanks Andreas the new patch applied straight away with no issues :)

Both the 'theoretical limit' issue and the 'too many threads' should be resolved.

Great :) Although whilst testing your latest patch the memory leak seems to be back :(

The test has been running for a few hours now and progressing very slowly with the JVM on the limit of the heap. Where as running the test with Jeremias' patch, the memory stays nice and lean well within the max heap and the documents are processed much faster.

No OOM Error yet, but I'm sure it can't be far away. I just wanted to update you before I go offline for the weekend.




Reply via email to