Github user rxin commented on a diff in the pull request:

    https://github.com/apache/spark/pull/1043#discussion_r13630905
  
    --- Diff: ec2/spark_ec2.py ---
    @@ -418,6 +418,12 @@ def launch_cluster(conn, opts, cluster_name):
             master_nodes = master_res.instances
             print "Launched master in %s, regid = %s" % (zone, master_res.id)
     
    +    # Give the instances descriptive names
    +    for master in master_nodes:
    +        master.add_tag(key='Name', 
value='spark-{cn}-master-{iid}'.format(cn=cluster_name, iid=master.id))
    --- End diff --
    
    Do you mind wrapping this before 100 char wide? We don't yet enforce it, 
but we will soon enforce line length in Python as well. Also for line 425. 
Thanks!



---
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.
---

Reply via email to