Github user shivaram commented on the pull request:
https://github.com/apache/spark/pull/2871#issuecomment-60121971
Yeah the only performance regression is in reading this volatile boolean if
you call `.value` many times. But I think that should be low (and it should be
insignificant if you call `.value` once which is I guess the most common use
case).
@rxin - I'll try to see if `transient lazy val` works. We'll still need
some logic to handle the `val` differently in the driver where we have to read
`obj` instead of `readBroadcastBlocks`
---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
