Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19893#discussion_r155366150
--- Diff: core/src/test/scala/org/apache/spark/SparkFunSuite.scala ---
@@ -52,6 +62,23 @@ abstract class SparkFunSuite
getTestResourceFile(file).getCanonicalPath
}
+ private def saveThreadNames(): Unit = {
+ beforeAllTestThreadNames =
Thread.getAllStackTraces.keySet().map(_.getName).toSet
+ }
+
+ private def printRemainingThreadNames(): Unit = {
+ val currentThreadNames =
Thread.getAllStackTraces.keySet().map(_.getName).toSet
+ val whitelistedThreadNames = currentThreadNames.
+ filterNot(s => SparkFunSuite.threadWhiteList.exists(s.matches(_)))
--- End diff --
style: `.filterNot { s => `
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
