This came up in CR: https://mongodbcr.appspot.com/591890009

We have some functions reporting errors by throwing where we at a higher level just use Status so we have to catch(...) and return exceptionToStatus()

It would be nice to clean up and rely more on exceptions.

Examples:
https://github.com/mongodb/mongo/blob/2f9f301e0247b059ba652c2488a450f1fbd1d77b/src/mongo/db/index/index_access_method.cpp#L547

https://github.com/mongodb/mongo/blob/2f9f301e0247b059ba652c2488a450f1fbd1d77b/src/mongo/db/catalog/validate_adaptor.cpp#L142