Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/20524#discussion_r166614859
--- 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 --
`assertNoOpenStreams` and `removeOpenStream` are both synchronized on
`openStreams` map. I want to increase interval to prevent starvation for
`removeOpenStream`.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
