Currently the internal architecture of BucketCatalog relies heavily on shared_ptrs and weak_ptrs. This leads to some confusing ownership semantics, and it becomes difficult to reason about object lifetimes. We should be able to make use of unique_ptrs in a single location, and raw pointers or references elsewhere as necessary.