Github user arunmahadevan commented on a diff in the pull request:

    https://github.com/apache/storm/pull/2801#discussion_r209442754
  
    --- Diff: storm-client/src/jvm/org/apache/storm/utils/ConfigUtils.java ---
    @@ -52,6 +79,16 @@ public static ConfigUtils setInstance(ConfigUtils u) {
             return oldInstance;
         }
     
    +    public static Map<String, Object> maskPasswords(final Map<String, 
Object> conf) {
    +        Maps.EntryTransformer<String, Object, Object> maskPasswords =
    --- End diff --
    
    @HeartSaVioR , thanks for the suggestion, we can replace 
STORM_ZOOKEEPER_TOPOLOGY_AUTH_PAYLOAD too.
    
    I was not aware of the "shaded-deps" module and its usage.
    
    Reverted to the earlier approach and using the shaded-deps now.


---

Reply via email to