[GitHub] [spark] dongjoon-hyun commented on a change in pull request #32287: [SPARK-27991][CORE] Defer the fetch request on Netty OOM

[GitBox]

dongjoon-hyun commented on a change in pull request #32287:
URL: https://github.com/apache/spark/pull/32287#discussion_r628807141



##########
File path: 
core/src/main/scala/org/apache/spark/storage/ShuffleBlockFetcherIterator.scala
##########
@@ -1005,4 +1118,11 @@ object ShuffleBlockFetcherIterator {
       address: BlockManagerId,
       e: Throwable)
     extends FetchResult
+
+
+  /**
+   * Result of fetch request that should be deferred for some reasons, e.g., 
Netty OOM

Review comment:
       `of fetch request` -> `of a fetch request`?




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org