gaussianrecurrence commented on pull request #699:
URL: https://github.com/apache/geode-native/pull/699#issuecomment-825235675


   > @echobravopapa I am not sure a deterministic test is reasonably achievable 
for this change. I think it would require a significant refactor to add hooks 
and metrics around this block of code just to maybe detect the other threads 
didn't pause for some meaningful time.
   
   Actually, I managed to write a test on linux by pausing the maintain 
redundancy threads. However thread naming is not supported until recent 
versions of Windows. So this test won't be available in windows. I agree with 
you @pivotal-jbarrett, I am not sure is worth it to put that much effort into 
making this test.


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

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to