Github user onursatici commented on the issue:
https://github.com/apache/spark/pull/21158
I think this PR is fine given it preserves the redaction of URL's. That was
indeed the main reason why url's were in the default redaction pattern in the
previous PR.
To keep all the previous behaviour, `SQL_OPTIONS_REDACTION_PATTERN` can
include `user`, as there might be environments considering user name
information on sql connection options as db credentials. Showing that
information could be opt-in if we redact by default.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
