bipinprasad opened a new pull request #3274:
URL: https://github.com/apache/storm/pull/3274


   ## What is the purpose of the change
   
   By default jvm disables assertion for performance, and we allow JVM options 
to be configured at deployment time, it is best to avoid use of assert 
statement and use explicit clauses to validate critical checks within 
storm-daemon code. Also, it would be helpful in many cases to explicitly log 
and proceed on failure to meet such assert expectation for debugging purpose.
   
   ## How was the change tested
   
   Rerun test suite in storm-server directory.


----------------------------------------------------------------
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:
us...@infra.apache.org


Reply via email to