tgravescs commented on pull request #30446: URL: https://github.com/apache/spark/pull/30446#issuecomment-736607828
sorry for my delay, behind on reviews. I think we should redact from both ui/rest or from event logging as people may not realize event logging has the password and history server could allow viewing from people who shouldn't see those. If people want it differently I would say we add a config to allow it but have it off by default - but I would wait until someone requests this. I think it was an oversight that these events weren't redacted. so if you see other cases we aren't we should fix them. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
