ashishkumar50 commented on PR #6780: URL: https://github.com/apache/ozone/pull/6780#issuecomment-2154310503
> > tmp is intended for special file system volume, but using sh it's allowed to do all the operation inside tmp volume which leads to discrepancies. > > @ashishkumar50 Are you suggesting that we don't allow "ozone sh" to delete buckets inside tmp? If bucket creation is allowed using sh then deletion also should be allowed inside tmp. I am okay with the change done in this PR to delete keys individually as some of them can't be accessed using fs commands and hence can't be deleted through fs command. Only concern here is user1 can't delete files inside tmp of other user2 file using fs command. But same user1 can delete files through sh command of other user2 file. If we want to allow tmp volume deletion better we can show some warning message here? -- 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]
