This is a rollup set of patches for modifications to TableIdentifier to have a getKey() which is now used for open_cache and table_share_def cache. This also includes additional unittests on TableIdentifier, and makes use of some cleanup, though not perfect, to TableShare for its contructors.