paulirwin opened a new pull request, #1131:
URL: https://github.com/apache/lucenenet/pull/1131

   - [X] You've read the [Contributor 
Guide](https://github.com/apache/lucenenet/blob/main/CONTRIBUTING.md) and [Code 
of Conduct](https://www.apache.org/foundation/policies/conduct.html).
   - [X] You've included unit or integration tests for your change, where 
applicable.
   - [X] You've included inline docs for your change, where applicable.
   - [X] There's an open issue for the PR that you are making. If you'd like to 
propose a change, please [open an 
issue](https://github.com/apache/lucenenet/issues/new/choose) to discuss the 
change or find an existing issue.
   
   Fixes the LimitedConcurrencyLevelTaskScheduler timing-dependent tests.
   
   Fixes #1123
   
   ## Description
   
   This PR should fix the two tests in TestLimitedConcurrencyLevelTaskScheduler 
that are timing-dependent and fail regularly in the constrained environment of 
Azure DevOps pipelines. This is likely because of thread contention, as these 
tests pass regularly locally and on GitHub actions. The two failing tests are 
the only tests in the class that depend on timing between the test thread and a 
parallel task, and an occasional failure should not generally be considered a 
full-suite failure as the test is nondeterministic in nature. So this changes 
the assertion to mark the test as Inconclusive if the values don't match at the 
time of evaluation.
   


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

Reply via email to