[ 
https://issues.apache.org/jira/browse/SPARK-8960?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14622147#comment-14622147
 ] 

Sean Owen commented on SPARK-8960:
----------------------------------

[~shivaram] [~nchammas] are you generally in favor of this idea or not so sure 
about it? I hadn't heard an objection to it and may free everyone up to work on 
the code more rapidly. Do you want to push on the separate repo idea? I'd 
support that.

> Style cleanup of spark_ec2.py
> -----------------------------
>
>                 Key: SPARK-8960
>                 URL: https://issues.apache.org/jira/browse/SPARK-8960
>             Project: Spark
>          Issue Type: Task
>          Components: EC2
>    Affects Versions: 1.4.0
>            Reporter: Daniel Darabos
>            Priority: Trivial
>
> The spark_ec2.py script could use some cleanup I think. There are simple 
> style issues like mixing single and double quotes, but also some rather 
> un-Pythonic constructs (e.g. 
> https://github.com/apache/spark/pull/6336#commitcomment-12088624 that sparked 
> this JIRA). Whenever I read it, I always find something that is too minor for 
> a pull request/JIRA, but I'd fix it if it was my code. Perhaps we can address 
> such issues in this JIRA.
> The intention is not to introduce any behavioral changes. It's hard to verify 
> this without testing, so perhaps we should also add some kind of test.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to