kerneltime commented on PR #3552: URL: https://github.com/apache/ozone/pull/3552#issuecomment-1167908774
> Thanks @kerneltime for updating the patch. I think moving the log statement to `putAcl` method would be more consistent, since error is also logged there, but this is good enough for fixing the bug. > > Also, please try to avoid force-push next time. There is a separate effort on going to not log customer generated messages into s3g log. The audit log should be at the higher level and this should be the right 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]
