Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/12157#issuecomment-209247352
I'm wary of giving equals semantics to partitions, since the current
semantics in this commit seem incorrect: partition 13 from one RDD is not equal
to partition 13 from another. Since it's not technically wrong to implement
`hashCode` without `equals`, seems like we can be conservative and not make
those changes. Adding `hashCode` is good, as is a style check if possible, as
are the other changes.
---
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]
