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

    https://github.com/apache/spark/pull/21158#discussion_r184152311
  
    --- Diff: 
core/src/main/scala/org/apache/spark/internal/config/package.scala ---
    @@ -342,7 +342,7 @@ package object config {
             "a property key or value, the value is redacted from the 
environment UI and various logs " +
             "like YARN and event logs.")
           .regexConf
    -      .createWithDefault("(?i)secret|password|url|user|username".r)
    +      .createWithDefault("(?i)secret|password".r)
    --- End diff --
    
    Not sure what you're trying to imply? The default is redacting too much. 
That needs to be fixed.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to