Add review comment for core/common/unordered
storage is terrible
factory should return shared_ptr