@sijie I agree with you.
But the specific case of id < 0 is fine to me, to throw NoSuchLedger because actually no ledger exists. In the (near) future we will have such cases of id < 0 so there no strong reason to change the behavior now. Okay to work on other validation cases. [ Full content available at: https://github.com/apache/bookkeeper/pull/1638 ] This message was relayed via gitbox.apache.org for [email protected]
