smengcl edited a comment on pull request #1021: URL: https://github.com/apache/hadoop-ozone/pull/1021#issuecomment-641465198
Some of the [bugs](https://sonarcloud.io/project/issues?id=hadoop-ozone&pullRequest=1021&resolved=false&types=BUG) that showed up in SonarCloud doesn't relate to OFS (e.g. `BasicOzoneFileSystem`/`BasicOzoneClientAdapterImpl`). ~~I will only be addressing the ones in OFS. i.e. `BasicRootedOzoneClientAdapterImpl` in HDDS-3767.~~ A closer look reveals that the bug isn't really a bug. If I "address" it by closing `ozoneOutputStream` in `finally`, the returning `OzoneFSOutputStream` will contain a stream that is already closed. ---------------------------------------------------------------- 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]
