Github user Parth-Brahmbhatt commented on the pull request:

    https://github.com/apache/incubator-storm/pull/190#issuecomment-49469937
  
    The simplest alternative seems to be no implementation for IAutoCredentials 
needed for AutoHDFS to work. In other words users will not  have to specify any 
class for "topology.auto-credentials" config for auto hdfs to work.
    
    User will specify AutoHDFS.java as "nimbus.credential.renewers.classes" and 
AutoHDFS will only implement ICredentialsRenewer. In the prepare phase of 
AutoHDFS.java, which should be called on nimbus startup, we can get the HDFS 
credentials.
    
    I have one clarifying question. The ICredentialsRenewer implementations 
seems to be loaded by reading "nimbus.credential.renewers.classes" config at 
startup by nimbus. If I understand correctly this means if we use 
ICredentialsRenewer the users who have a running nimbus and wants to use 
AutoHDFS will have to change the config and restart the nimbus. Is that 
acceptable? 
    
    



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