Hi devs, While debugging storm-autocreds issue, I had hard time to follow the code path since the class implements two different kinds of interfaces together.
Auto* classes implement three interfaces for two kinds of purposes: - INimbusCredentialPlugin, ICredentialsRenewer (Nimbus plugin) - IAutoCredentials (worker, topology submitter) and it shares implementation of prepare(). I've already prepared PR for proposal: https://github.com/apache/storm/pull/2161 Please note that I also removed cluster-wide configurations, since it was also confusing me, and may open potential security vulnerability. (Even small code change could open it) This breaks backward compatibility, so would like to see everyone is OK to. Would like to hear your thought. Thanks, Jungtaek Lim (HeartSaVioR)
