mridulm commented on code in PR #36512: URL: https://github.com/apache/spark/pull/36512#discussion_r876759477
########## core/src/main/scala/org/apache/spark/storage/BlockManager.scala: ########## @@ -933,10 +933,29 @@ private[spark] class BlockManager( }) Some(new BlockResult(ci, DataReadMethod.Memory, info.size)) } else if (level.useDisk && diskStore.contains(blockId)) { - try { - val diskData = diskStore.getBytes(blockId) - val iterToReturn: Iterator[Any] = { - if (level.deserialized) { + var retryCount = 0 + val retryLimit = 3 Review Comment: That sounds reasonable to me. My only concern is, if this is some jitter/transient issue, we will end up recomputing stage(s)/blocks - though I have not checked this in our clusters - +CC @otterc have you observed this behavior ? +CC @Ngone51 as well for your thoughts. -- 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. To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org 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