SPARK-22479 was filed to not print the log jdbc credentials, but in there they also added the username and url to be redacted. I'm not sure why these were added and to me by default these do not have security concerns. It makes it more usable by default to be able to see these things. Users with high security concerns can simply add them in their configs.
Also on yarn just redacting url doesn't secure anything because if you go to the environment ui page you see all sorts of paths and really its just confusing that some of its redacted and other parts aren't. If this was specifically for jdbc I think it needs to be just applied there and not broadly.
If we remove these we need to test what the jdbc driver is going to log from