Untangle snapshot mapper

Also fixed performance issue with `StructEncoding` arbitrary
SnapshotFactsetBaseMapper doesn't make sense any more given there is only one thing that extends it. There is inheritance and a whole bunch of hoop jumping (and hiding spots for bugs) that can be elliminated by just making a single correct mapper.