Github user aarondav commented on the pull request: https://github.com/apache/incubator-spark/pull/613#issuecomment-35432325 LGTM too after @rxin's change. Since Double is used throughout that file, I think renaming JDouble is cleanest.
--- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. To do so, please top-post your response. 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. ---