Github user tgravescs commented on the pull request:

    https://github.com/apache/spark/pull/29#issuecomment-36253998
  
    Thats really unfortunately that hadoop 1.x doesn't support it, as I would 
prefer to use the addCredentials since it also handles secrets.    
    
    Our only option right now would be to add a separate function in 
SparkHadoopUtils and YarnSparkHadoopUtils. This would allow spark on yarn to 
use the addCredentials at least.  Maybe its not critical right now. thoughts?
    
    Otherwise changes look good.


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

Reply via email to