mysql 8.0 reports DESC etc. for columns in indexes

[sqlalchemy-bot]

Migrated issue, originally created by Michael Bayer (@zzzeek)

Postgresql dialect supports this, so for mysql8.0 we have to ramp up the parsing:

from sqlalchemy import *

m1 = MetaData()

t1 = Table(
    'add_ix', m1, Column('x', String(50)), mysql_engine='InnoDB')
Index('foo_idx', t1.c.x.desc())

for url in [
    "postgresql://scott:tiger@localhost/test",
    "mysql://scott:tiger@mysql57/test",
    "mysql://scott:tiger@mysql80/test",
]:

    e = create_engine(url)
    m1.drop_all(e)
    m1.create_all(e)

    insp = inspect(e)
    assert \
        insp.get_indexes("add_ix") == \
        [{'name': 'foo_idx', 'unique': False, 'column_names': ['x']}], \
        "failed on %s" % url

[sqlalchemy-bot]

Michael Bayer (@zzzeek) wrote:

Reflect ASC/DESC in MySQL index columns

Fixed bug in index reflection where on MySQL 8.0 an index that includes
ASC or DESC in an indexed column specfication would not be correctly
reflected, as MySQL 8.0 introduces support for returning this information
in a table definition string.

Change-Id: I21f64984ade690aac8c87dbe3aad0c1ee8e9727f
Fixes: #4293

→ 9d2dc79

[sqlalchemy-bot]

Changes by Michael Bayer (@zzzeek):

• changed status to closed

[sqlalchemy-bot]

Michael Bayer (@zzzeek) wrote:

Reflect ASC/DESC in MySQL index columns

Fixed bug in index reflection where on MySQL 8.0 an index that includes
ASC or DESC in an indexed column specfication would not be correctly
reflected, as MySQL 8.0 introduces support for returning this information
in a table definition string.

Change-Id: I21f64984ade690aac8c87dbe3aad0c1ee8e9727f
Fixes: #4293
(cherry picked from commit 9d2dc79)

→ 2d5e206