tgravescs commented on pull request #28323: URL: https://github.com/apache/spark/pull/28323#issuecomment-620001049
it's been a long time since I looked at this so I will have to go refresh my memory but I don't think using getUserName works in all cases. What kind of testing have you done on this? Note the question is How was this patch tested, so can you please give more details on unit tests and manual testing done. In this case we need to have tested in various secure environments. ---------------------------------------------------------------- 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]
