spark-ec2 icon indicating copy to clipboard operation
spark-ec2 copied to clipboard

Moves from optparse to argparse

Open thisisdhaas opened this issue 8 years ago • 2 comments

See https://docs.python.org/2.7/library/argparse.html#upgrading-optparse-code for a checklist of things that should have been done (which I tried to follow), and http://stackoverflow.com/questions/3217673/why-use-argparse-rather-than-optparse for a discussion of why this is a good idea in general.

See #26 for a discussion of dropping support for python 2.6 in spark-ec2 2.0

thisisdhaas avatar Jun 14 '16 22:06 thisisdhaas

Thanks @thisisdhaas - Change mostly LGTM. One thing is that it would be good to change the README to note that we require python 2.7 from here on

cc @nchammas

shivaram avatar Jun 14 '16 23:06 shivaram

Didn't test it out, but the changes LGTM.

And yes, the README should make it loud and clear that we've dropped Python 2.6 support. Better would be to capture that in code somehow, so the error when someone runs on 2.6 is clean, but a README note is good enough IMO.

nchammas avatar Jun 15 '16 14:06 nchammas