Github user ankurdave commented on the pull request:
https://github.com/apache/spark/pull/2439#issuecomment-56258349
@rnowling Hmm, maybe you're right about that -- I'm not familiar enough
with the algorithm to know whether it specifies rounding behavior in the first
place.
I think the unit test you wrote does do the [range
check](https://github.com/apache/spark/blob/master/graphx/src/test/scala/org/apache/spark/graphx/util/GraphGeneratorsSuite.scala#L68).
In fact that's how I found this bug in the first place: the test failed in
Jenkins in an unrelated 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]
