All of these modulestore type checks make me wonder if the intent is to distinguish read-only from read-write stores. If so, then it seems we should expose that as a characteristic rather than 'split', 'mongo', 'xml' or whatever. Just wondering.