Github user dorx commented on the pull request:
https://github.com/apache/spark/pull/1025#issuecomment-46402513
@srowen Hey Sean, turns out Colt(or breeze for that matter) doesn't have
inverseCDF for the distributions, which I need for the implementation. The
original thought was to avoid increasing the number of dependencies for Spark,
as we need to support multiple build frameworks (and it's a lot of
compatibility testing work for the person merging the PR). As it turns out,
there isn't really a viable substitute in the current dependencies, which was
why I went and got the approval for adding commons-math3 as a dependency.
---
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.
---
