hemantk-12 commented on PR #6012: URL: https://github.com/apache/ozone/pull/6012#issuecomment-1894612792
> Thanks for the improvement @jianghuazhu. I think we should add the key rename check in the OMDirectoryCreateRequest as well. It looks like this check was originally added before the file system optimization feature. > > Also, I think we should we add a call to the snapshot reserved word check inside the new util method (regardless of the config value) to apply that check to all keys and directories in one spot. @hemantk-12 does that make sense? It makes sense to have keyName validation at one place. -- 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]
