hasnain-db opened a new pull request, #43596:
URL: https://github.com/apache/spark/pull/43596

   ### What changes were proposed in this pull request?
   
   Improve a few timing related constraints:
   
   * Wait 10s instead of 5 for a reload to happen when under high load. This 
should not delay the test in the average case as it checks every 100ms for an 
event to happen.
   * In certain cases we run *too fast* so the new file we create has the same 
timestamp as the old file, and thus we never reload. Add a sleep there so the 
modification times are different. This was accidentally reverted in 
https://github.com/apache/spark/pull/43249/commits/b7dac1f96ec45a300963e4da1dc4fc1173470da7#diff-de96db6b61e9f48fb9bd8b781f4367e60a48b3886dfe03d4cf16b47ef6c26d0a
   
   ### Why are the changes needed?
   
   These changes are needed to make the test more reliable and less flaky under 
load.
   
   
   ### Does this PR introduce _any_ user-facing change?
   
   No
   
   
   ### How was this patch tested?
   
   Ran this test in parallel on a machine under high load. Previously under 
those conditions I would repeatedly get high rates of failure (80%+) and now it 
does not fail.
   
   
   ### Was this patch authored or co-authored using generative AI tooling?
   
   No
   


-- 
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: [email protected]

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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to