[GitHub] storm pull request #2801: STORM-3184: Mask the plaintext passwords from the ...

[HeartSaVioR]

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

    https://github.com/apache/storm/pull/2801#discussion_r209424838
  
    --- 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 --
    
    Could we replace this with Java streams API and remove Guava dependency?


---