Segun-Ogundipe commented on pull request #4299:
URL: 
https://github.com/apache/incubator-dolphinscheduler/pull/4299#issuecomment-755452907


   Hi @CalvinKirs, the 3 remaining bugs are from the DAG file in the common 
project. Three fields are declared with the `volatile` keyword and they are 
what sonarcloud tags as bugs. Can I remove the `volatile` keyword or is there a 
better way to make it work?
   
   I'm also trying to increase the code coverage to get it over 33% but there 
are some tests that are ignored are not letting the codes I added to get 
covered. Please advice about what to do here.
   
   The 53 vulnerabilities are from files I refactored to fix code 
vulnerabilities previously reported by sonar. It stills complain about them 
even though I have fixed them. Is there a way for me to fix this?
   
   I'll really appreciate your guidance here. Thanks.


----------------------------------------------------------------
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]


Reply via email to