Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/6735#issuecomment-110731067
Heh, well it's supposed to be used when in the `spark-shell` I suppose.
However I agree that it seems to have not been used for a long time,
accidentally.
I don't know the history of this bit of code. I am not sure about the
assumption in the comments that the constructor has no effects -- did it mean a
case class? It sort of looks like an optimization, to try to call the
constructor directly via reflection.
Well... I think it wouldn't hurt to try removing it as you say and see what
happens?
---
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]
