I'd like for us to avoid having public methods in the reader that return ColumnList. The only reader method that returns a ColumnList and is called outside of tests is `getAllLocators`, and it should probably be changed to return List<Locator>.

There's also already a method on the reader that has almost identical behavior (getMetadataValues). We should have just one method for getting metadata values for a locator.