jojochuang commented on PR #9502: URL: https://github.com/apache/ozone/pull/9502#issuecomment-3658040710
@ivandika3 oh yeah that can break for /tmp path manipulation. Hmm this is awkward. We would need to force OFSPath to accept a user name as a parameter, and the OFSPath usage is everywhere. I am thinking to add another OFSPath constructor to accept the "initialization user name", and abort if it turns out the path is a /tmp and that the initialization user name is not set. It's going to be awkward and error prone still. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
