Github user vanzin commented on the pull request:

    https://github.com/apache/spark/pull/2449#issuecomment-56099817
  
    Hi, mostly so I understand what's going on (I've traced the code but it's 
kinda hard to know if I covered everything), how are jars distributed in yarn 
cluster mode after your change?
    
    It seems to me that your change just makes the `--jars` command line option 
be ignored for yarn cluster mode. If that's the case, I'm not sure I like it so 
much...
    
    If that's not the case, could you clarify in the PR description how 
`--jars` works in yarn cluster mode? That would help others who might want to 
understand why this change works in spite of looking counter-intuitive.


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to