Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/22371#discussion_r216166979 --- Diff: core/src/main/scala/org/apache/spark/shuffle/IndexShuffleBlockResolver.scala --- @@ -51,6 +52,8 @@ private[spark] class IndexShuffleBlockResolver( private val transportConf = SparkTransportConf.fromSparkConf(conf, "shuffle") + private val shuffleIdToLocks = new ConcurrentHashMap[Int, Array[Object]]() --- End diff -- How about `Array[_]` if the type is truly irrelevant? But I get that it's always going to be an Object given the usage here.
--- --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org