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

   This fixes an issue with a slow test in TestTaxonomyCombined. The original 
issue description is here:
   
   The issue has been brought up in 
https://github.com/apache/lucenenet/issues/836, under 
`TestTaxonomyCombined.TestTaxonomyRefreshRaces()` section
   
   After debugging the issue, I discovered that the cause of the slow down are 
System.Thread.Sleep(1) calls being made thousands of times with intent of sleep 
taking 1ms. On windows however, that could take much more than 1ms, 15ms 
normally, confirmed locally and found a discussion about it here:
   
   
https://stackoverflow.com/questions/19066900/thread-sleep1-takes-longer-than-1ms
   
   I setup Java version locally to compare the running times and after the fix, 
Java version and .NET version is running in basically identical amount of time. 
Locally I get it to run in 3-4s range, the same with Java version. Before the 
fix, it took 30-40s depending on what random generated loop variable values 
were.
   


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