The getAllShards function in the catalog client returns a StatusWith which generally implies that it won't throw a DBException and will instead return any errors as a status. However, we assert that the find command worked, causing places which aren't expecting this function to throw to fail.

In line with the current recommendations on exceptions and statuses, we should just convert this function to throw always rather than return a status.