FIx #17933: Superset fix chart query for mysql con.

[harshsoni2024]

Describe your changes:

Fixes #17933

I worked on ... because ...

Type of change:

• Bug fix
• Improvement
• New feature
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Documentation

Checklist:

• I have read the CONTRIBUTING document.
• My PR title is Fixes <issue-number>: <short explanation>
• I have commented on my code, particularly in hard-to-understand areas.
• For JSON Schema changes: I updated the migration scripts or explained why it is not needed.

[IceS2]

Looks good, but maybe we could have the logic in one place, doing something like:

def getSupersetQuery(String query, Connection connection):
  if isinstance(connection, MysqlConnection):
    return query.replace('"', "`")
  return query

[sonarqubecloud]

Quality Gate passed for 'open-metadata-ingestion'

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
55.6% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

[harshsoni2024]

Looks good, but maybe we could have the logic in one place, doing something like:

def getSupersetQuery(String query, Connection connection):
  if isinstance(connection, MysqlConnection):
    return query.replace('"', "`")
  return query

I'll do this in next changes sure. Need to cherry-pick this branch today 😅