Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/8314#issuecomment-135960281
@mengxr commons math 3.x is already a dependency in core. I don't have
benchmarks handy, but my experience with the RNGs is that they're at least
"more than fast enough" for any purpose I've had. I don't think the RNGs are
changing, and they implement particular RNG processes like Well19937 that
should not change over time. The down-side to not using it is simply the higher
probability of bugs, like this one, when implementing from scratch.
I think we need to get this fix in in any event. @squito do you need a
hand? I think simply loosening the tests is appropriate, even if it means
bigger tests.
---
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]
