Github user shivaram commented on the pull request:
https://github.com/apache/spark/pull/3986#issuecomment-71289144
@nchammas @GenTang - The `logging.basicConfig` seems to have been around
since the very beginning [1]. I don't know much about Python so I can't
recommend keeping it or removing it. @JoshRosen can comment on that.
Other than that this solution looks fine to me. It is unfortunate that we
have so many custom sleep calls across the file, but I don't think there is
much else we can do given the EC2 API we have right now.
[1]
https://github.com/mesos/spark/blob/08c50ad1fcf323f62c80dfeb8f1caaf164211e0b/ec2/spark_ec2.py#L538
---
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]
