Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20546#discussion_r167071776
--- Diff: core/src/main/scala/org/apache/spark/storage/StorageUtils.scala
---
@@ -46,7 +42,7 @@ class StorageStatus(
* Internal representation of the blocks stored in this block manager.
*
* We store RDD blocks and non-RDD blocks separately to allow quick
retrievals of RDD blocks.
- * These collections should only be mutated through the
add/update/removeBlock methods.
+ * These collections should only be mutated through the addBlock method.
--- End diff --
I think this is pretty out of date now. I don't see any calls to `addBlock`
outside of this class.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
