remove comment that is a lie

Change-Id: Iee285608cfec2b9ab1d122597138e5a1148cdf36
Be more efficient

No sense filling up the hash and then wiping it out later if $table_name is set

Change-Id: Ie81b55e349ef71103cd83354d220b890fa37c0d3
Remove duplicate variable $data_source -> $self

This is left over from when this function lived in UR::Namespace::Command::UpdateClasses

Change-Id: I55ca7f93cd70b8a130fac64a4dc247873d4eb093
Extract method to get or create the table object during metadata refresh

Change-Id: I07f436d6dda093dabe3a79cb21e32a4c4cfa6358
Extract method to update column metadata during metadata refresh

Change-Id: Ic0cd0e531b792e3ff29fd537404e55f25f53ce89
remove unused code for detecting bitmap indexes

Change-Id: I10a46398b91c964b6a0f9daa605d9794b31529b8
Extract method to update foreign key metadata during refresh

Change-Id: I2b07fd94a7b59435ef77616389ce7d821a0ebe4a
Extract method to update primary key metadata during refresh

Change-Id: I55b56ed73ce0bbbb99ce28d390d99303e26a670f
Extract method to update unique constraint metadata during refresh

Change-Id: Ia5e34d2dfee6367d93067e611cdae231b4fe75f5
Use fetchrow_hashref() instead of fetch() with var bindings

Change-Id: I93c8b7a3968239963ab31cf00ffe497aa712894e
_get_table_names_from_data_dictionary() can return qualified table names

If the data source has no 'owner', then it looks for tables in all attached
schemas and returns qualified table names including the schema (schema.table)

Change-Id: I2e4d608d388c895f477d9bc4c48b06f7266ef65f
extract method to determine what table name to use for DB metadata objects

Change-Id: Ia9c042b5603eedb9bdda1f963cdee3bc8cbba462
Remove ancient commented-out code

Change-Id: Icb611cae33b734bb20a0a624fc7083f48c21c23d
Remove owner from the primary key of all the MetaDB tables

Change-Id: Ieb66e40a9dfff755214cb794c97c9bbe31a56c1c
SQLite now returns schema info when querying for foreign keys

Change-Id: I5f656f4f46adf20f136a0ba2780e15b6853b064d
'owner' is no longer tracked in the MetaDB or its objects

If a data source only wants to track a single schema, it should have a value
for its 'owner' property/method.  In this case, class' table names will be
simple table names.

If a data source wants to track all schemas in a database, it should return
a false value from its 'owner' property.  Class' table names will be strings
like schema.table.

The 'owner' column in the metaDB is no longer a primary key in any table
and is no longer getting filled in.  We can drop the columns later when we
have a plan for migrating the MetaDB.
Forgot use strict/warnings

Change-Id: Iea45422f528067cb9a73c36e688a5ae79c380cfe
Avoid an uninitialized warning in _table_name_to_use_for_metadata_objects()

If we didn't find a table object using the bare table name, and the data
source has an owner (meaning it normally wouldn't be storing qualified
table names in the MetaDB), then try looking it up as a qualified name.

This avoids the warning because looking up a non-existent table object
would fall through to the second get()

Change-Id: Ib3628302efdbffa9a57a642e2d99722292837d7a
Allow constructs like schema.table as an argument

To get table info for other attached DBs/schemas

Change-Id: Ied089e713357d2937dfaa149cd1c1fdcf2ea33d7
SQLite dumps shouldn't include schemas other than 'main'

Change-Id: I8c302388220545c10d279b04cb3a0d63ec87b55a