NightOwl888 commented on issue #886: URL: https://github.com/apache/lucenenet/issues/886#issuecomment-1822964001
@Jeevananthan-23 - https://github.com/apache/lucenenet/commit/963e10ca259570451f953331e4a9d62fabaffa32 points to #325 where the [original error report](https://github.com/apache/lucenenet/pull/325#issuecomment-671766858) is. I followed up with [another stack trace on the same failing test](https://github.com/apache/lucenenet/pull/325#issuecomment-672154841). I have reverted the relevant changes from 963e10ca259570451f953331e4a9d62fabaffa32 in this branch: https://github.com/NightOwl888/lucenenet/tree/fix/documentswriter-concurrency. I ran the tests 30 times on Azure DevOps and ran the `TestMultiThreadedSnapshotting` test locally 30,000 times and couldn't get a failure. That is the good news. The bad news is that another test `TestRollingUpdates.TestUpdateSameDoc` fails, but very rarely. I got it to fail locally on both .NET 5.0 and .NET 6.0, but not on .NET 7.0. So, we cannot merge the patch until we have a fix for the failing test. I am attaching the log from the test failure. I got it to fail on `net5.0` on Windows (the [original failure](https://dev.azure.com/lucene-net-temp2/Lucene.NET/_build/results?buildId=969&view=ms.vss-test-web.build-test-results-tab&runId=642992&resultId=100400&paneView=debug) was on Linux). I used the `[Repeat(1000)]` attribute on the test, and it failed after about 3 runs. I also used the assembly attributes as specified in the test failure. This ensures the same random components are plugged into the test during each run, which may help narrow down which component is faulty. On the other hand, these may have nothing to do with the exception at all - it is hard to determine this when the failure happens so rarely. Do note we have our own random class so these will work consistently across target frameworks and operating systems. ```c# [assembly: Lucene.Net.Util.RandomSeed("0xe6dee1082501680d")] [assembly: NUnit.Framework.SetCulture("sat-Olck")] ``` [TestUpdateSameDoc-638362856625826917.zip](https://github.com/apache/lucenenet/files/13440868/TestUpdateSameDoc-638362856625826917.zip) If you could pull down the branch to investigate why the test is failing, that would be great. `TestTargetFramework.props` is where the target framework for the tests can be specified. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@lucenenet.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
