apache / apache/bookkeeper

Remove asserts from code

Open
#995 0 comments 0 reactions 0 assignees View on GitHub
triage/week-33
Dominant language
Java
Stars
2k
Forks
976
Avg merge
6d 15h
Merged PRs (30d)
7

Description

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.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.