Prevent introspection errors from crashing inspectdb

[Adam Vandenberg]

If you're inspecting a MySQL db, and it has a view that references missing columns, then inspectdb will stop with an exception while inspecting that database.

Add a "--skip" option to inspectdb to skip over tables that throw errors instead of aborting.

[Anssi Kääriäinen]

The first impression I get from --skip is that you can list tables you don't want to inspect. Better name could be something like --ignore-errors?

Another solution would be to provide --skip-tables and --only-tables, so that one could manually skip the error producing tables.

[Adam Vandenberg]

Updated patch.

[patchhammer]

skip.diff fails to apply cleanly on to trunk

[Adam Vandenberg]

If there is still interest in accepting this, I can rework the patch; otherwise a maintainer ought to close it.

[Claude Paroz]

Output errors

[Claude Paroz]

The attached patch, instead of adding a new command parameter, just output in the result that some table was not introspected. For example:

# Unable to inspect table v_test
# The error was: View 'test_.v_test' references invalid table(s) or column(s) or function(s) or definer/invoker of view lack rights to use them

[Claude Paroz]

​PR

[Claude Paroz <claude@…>]

In 4c18a8a3:

Fixed #14098 -- Prevented crash for introspection errors in inspectdb

Thanks Tim Graham for the review.