READ COMMITTED requirement check doesn't account for database views

Bumping this to major. This is very confusing for site owners. Would be great if we could fix this before Drupal 10.1 is released. Maybe this should even be a "critical" one.

The issue only applies to MySQL/MariaDB views. Right?

Yes

@chi you are right.

I think it would be reasonable to restrict ::findTables() to only finding actual database tables - we could do that with a change record probably.

I tried to think of bc-friendly options (adding a second argument for $include_views = TRUE? adding a settings override?) but couldn't come up with a use case.

I guess this just needs someone to write a patch?

Here's a patch to get things started. This will:

1. Add an optional parameter to findTables() to exclude SQL views from the query.
2. Use that parameter in the mysql_requirements() call in mysql.install.

The patch works correctly on a 10.1.6 drupal site and stops showing warnings against views in the database.

Confirming this patch worked for me on 10.1.6 as well.

Confirm this works for me too.

Confirming that this worked OK with 10.2.