You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@ozone.apache.org by GitBox <gi...@apache.org> on 2022/06/02 11:28:02 UTC

[GitHub] [ozone] ChenSammi commented on pull request #3474: HDDS-3630: [Merge rocksdb in datanode] merge feature branch to master

ChenSammi commented on PR #3474:
URL: https://github.com/apache/ozone/pull/3474#issuecomment-1144754960

   There are three issues reported by Sonar.  We will leave them be. 
   1. Medium security issue caused by using "ThreadLocalRandom."  Since we leverage the random number to choose which DbVolume to use. It's not security-sensitive code. So we will leave it be. 
   2. Use try-with-resources or close this "IngestExternalFileOptions" in a "finally" clause.  False alert. 
   3. Use try-with-resources or close this "Options" in a "finally" clause.  Actually this Options class doesn't have a close function. 
   
   


-- 
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: issues-unsubscribe@ozone.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org