Github user shivaram commented on the pull request:
https://github.com/apache/spark/pull/134#issuecomment-41602709
@logc -- Sorry I am caught with with a deadline and didn't get a chance to
look at this. I think the basic idea is that we don't want users to run
setup.py to use spark_ec2.py. This script is meant to be a utility script that
just works out of the box without any installation. This is part of the reason
why we ship the dependencies and include them in the PYTHONPATH in the shell
script `spark-ec2`.
Anyways I think this change looks good. We could add a `run-tests`method
that will put moto and mock in the PYTHONPATH, but I am not sure that is very
necessary as these tests won't be executed by all users.
@pwendell can you take a look before merging ?
---
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.
---
