This is an improvement to DM-29737. This include a log output stating that the catalog is not sorted and that finding a record by key may be slow. Moreover, DM-29737 did not adhere to C++ style guides. This ticket fixes that issue.
Jenkins run: https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/34026/pipeline
Looks good! Only change request is to make the logging DEBUG level.