Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/11436#issuecomment-190937142
@JoshRosen This looks good, but still somewhat complex. In particular,
there are a few unintuitive signatures in `BlockManager` and related classes
that I think can be improved. I made a suggestion for `doPut`'s return type,
but I'm still thinking about how to avoid the `Either` in `getOrElseUpdate`'s
return type. I think this task is inherently complex so eventually we'll run
out of things to simplify without a significant refactor.
---
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]
