Github user vanzin commented on the pull request:

    https://github.com/apache/spark/pull/1218#issuecomment-50417105
  
    Hi Andrew,
    
    Thanks for looking at this; it will take me some time to context-switch 
these back in and make the necessary updates, though.
    
    Regarding my open PRs, SPARK-2169 is independent of the others and should 
be pretty straight-forward. The other two don't really have an order - they 
just conflict with each other, so once one is in I'll need to fix the other.
    
    Regarding the "app id" question, I think the app ID is a cluster-specific 
concept. I think it's a little sketchy to let the framework create some ID when 
the cluster manager doesn't do it - and I'm especially wary of using timestamps 
for that. It's probably better to let the affected parties (e.g. the history 
server) decide what's the best approach when an application does not have a 
cluster-provided ID.


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