Github user nchammas commented on the pull request:

    https://github.com/apache/spark/pull/1344#issuecomment-48514129
  
    > If users want to select all spark clusters, can't they just name their 
clusters spark-XXX? 
    
    Absolutely, and I think that's fine. 
    
    > I would worry a bit about changing the naming now because I know people 
automate things around these ec2 scripts and this would introduce a change that 
is not backwards compatible since we'd coerce spark- into the name.
    
    Yes, that is my concern too, which is why I opted for removing the `spark-` 
from the instance tags as opposed to adding it in to the security group names. 
    
    I originally [made the change](https://github.com/apache/spark/pull/1043) 
to add instance name tagging in, and it's [not in the 1.0.0 
release](https://github.com/apache/spark/blob/branch-1.0/ec2/spark_ec2.py#L395),
 so I think it's safe to change the instance name tags as long as it makes it 
in to the next release. 
    
    One way or the other, the next release will be the first where `spark-ec2` 
tags instances with names.


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