You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We currently assert in production code in a few places. These are disabled at runtime, and so have little value. We should replace these with thrown exceptions, or crash the bookie if they are serious enough.
We should also add a checkstyle rule to forbid them.
The text was updated successfully, but these errors were encountered:
### Motivation
See #995
### Changes
1. use junit assert instead of java assert statement in test
2. delete a unused util class also uses assert statement
Signed-off-by: ZhangJian He <shoothzj@gmail.com>
We currently assert in production code in a few places. These are disabled at runtime, and so have little value. We should replace these with thrown exceptions, or crash the bookie if they are serious enough.
We should also add a checkstyle rule to forbid them.
The text was updated successfully, but these errors were encountered: