Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/7268#issuecomment-121703386
Thanks for doing this cleanup. I agree that the `BlockStoreShuffleFetcher`
doesn't really serve much purpose anymore, since it's really just providing a
single convenience constructor / wrapper for `ShuffleBlockFetcherIterator`
(especially given that it's the only non-test code that creates
`ShuffleBlockFetcherIterator`).
I'm going to take a pass through this now. I have a couple of questions
about some of the MapStatus conversions that we perform, since I've always
found this code to be a little tricky. This patch cleans things up
significantly, but there's still a few points I'd like to clarify so I'll
comment inline.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
