Github user derrickburns commented on the pull request:
https://github.com/apache/spark/pull/2419#issuecomment-57404998
@mengxr
1. I fixed the merge issue and also remerged to capture more recent changes.
2. I did as you suggested and introduced a local variable to hold a value
used in the closure. *Unfortunately, the test still fails. Since all values
used in the closure are local variables, I do not know what to do at this
point.*
3. The best fix to the problem with the test that selects two random
centers is the one that I implemented. It will pass the test
deterministically, while still using any random seed.
---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
