add option to re-apply naming conventions for table renamed in batch mode

[zzzeek]

We thought this was a bug which we can see in discussion at #1833. however, from the point of view of "batch migrations" as a workaround for the lack of ALTER, it's not a bug.

1. Batch migrations recreate a table with a new name, then drop the old table and rename the new table to the old one
2. it normally uses reflection to get the full structure of the existing table, including all of its constraints (and their names). that is, the constraints that are already created in the database.
3. a SQLite bug that was fixed in SQLAlchemy 2.0.45 sqlite table names that screw up the reflection regex sqlalchemy#12924 had the behavior such that SQLite constraints where the constraint name used MixedCase would not reflect the constraint name; we'd get None for the constraint name in that case
4. when batch reflection / table recreation gets None for a constraint name, it uses the configured naming convention (if any) to give the new constraint a name
5. user @mbergen had a schema in SQLite using a naming convention where constraints had MixedCase.
6. up through SQLAlchemy 2.0.44, @mbergen relied upon the behavior that a batch migration for SQLite where the table name changed would also rename their constraints for that new table name, so a constraint called "Old_Table_ck_1" would nicely become "New_Table_ck_1" (noting MixedCase names)
7. the bug fixed in 2.0.45 meant the operation would now successfully reflect the constraint's name of "Old_Table_ck_1" and leave it unchanged when creating a new table.
8. the behavior for constraint names with lowercase on SQLite as well as PG, MySQL was always that the constraint name was reflected, and applying a table rename and/or a batch recreate didnt change those names.
9. this behavior also matches what happens if we use ALTER TABLE. If you have a table named "Old_Table" and your naming convention made a constraint "Old_Table_ck_1", and you do "ALTER TABLE 'Old_Table' RENAME TO 'New_Table'", that's not going to rename your "Old_Table_ck1" constraint - that old name stays.
10. this all means the current behavior plus the 2.0.45 sqlite fix for alembic is "correct", for a "correct" of "batch migrations are a substitute for ALTER TABLE".

So. With all of the above establishing this was not a bug, the batch mode thing does give us a way to...rename a table and drop and recreate all its constraints! so maybe it would be nice for there to be an option, "rename_constraints_with_convention=" or something like that, so that you can actually have that behavior @mbergen had come to rely upon.

that's the issue here. feature request

[sqla-tester]

Mike Bayer has proposed a fix for this issue in the main branch:

apply naming_convention to reflected constraints in batch recreate https://gerrit.sqlalchemy.org/c/sqlalchemy/alembic/+/6832