Github user tdas commented on the pull request:
https://github.com/apache/spark/pull/6614#issuecomment-109045225
This is getting a little bit complicated to think how this will work for
all combinations of ReceivedBlock types, and StorageLevels and
ReceivedBlockHandler types. This must be done with unit tests. Mind adding unit
tests for this A x B x C combinations in the ReceivedBlockHandlerSuite, so that
we can clearly verify for what combinations we cant do this, and what
combinations we do it correctly?
---
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]
