@maskit it does need to be in the same package because it's not a clean mock like we would have if just mocking an interface. It relies on some package private members of the classes it is overriding unfortunately. If we can move managed ledger to completely using the new bookkeeper apis, then the mocks can go in whatever package we want.
[ Full content available at: https://github.com/apache/incubator-pulsar/pull/2544 ] This message was relayed via gitbox.apache.org for [email protected]
