storm-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Parth-Brahmbhatt <...@git.apache.org>
Subject [GitHub] incubator-storm pull request: STORM-346: added AutoHDFS class that...
Date Fri, 18 Jul 2014 19:24:18 GMT
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message