Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/20524#discussion_r166699026
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/test/SharedSparkSession.scala ---
@@ -111,7 +111,7 @@ trait SharedSparkSession
spark.sharedState.cacheManager.clearCache()
// files can be closed from other threads, so wait a bit
// normally this doesn't take more than 1s
- eventually(timeout(10.seconds)) {
+ eventually(timeout(10.seconds), interval(2.seconds)) {
--- End diff --
The default is 15 milliseconds. It should be fine.
But, I am also fine to change this to 2 seconds. I checked the test history
and this does not increase the testing time.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
