Github user tbertelsen commented on the pull request:
https://github.com/apache/spark/pull/4534#issuecomment-74246192
It looks like it fixes the issue with `Seq()`, but not e.g. `new Array(2)`
(have not run it though). But after our discussion I think that is perfectly
fine.
If you manage to create a non-empty collection of `Nothing`, then
practically nothing will work in spark, or in scala for that matter, e.g.,
`(new Array(2)).toList` won't compile.
Handling `RDD[Nothing]` seems like it will require quite some work, but I
fail to see the use case where it brings real functionality.
In summary: your fix seems like a good solution. I'll close this PR.
---
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]
