fix: Add postgres compatibility in PrestoViewMetadataExtractor

[chonyy]

Summary of Changes

• Related issue: Hive Metastore PostgreSQL compatibility issue on extractors #1608
• Currently, PrestoViewMetadataExtractor is only workable with MySQL connection string. The syntax of the SQL statement is not compatible with Postgres
• Followed the method in HiveTableMetadataExtractor
  • Added SQL statement for Postgres
  • Added a helper function to choose the SQL statement based on the connection string

Tests

This change has been running in our production environment for a couple of months. I believe no additional unit test is needed since it's simply fixing an unexpected behavior.

CheckList

Make sure you have checked all steps below to ensure a timely review.

• PR title addresses the issue accurately and concisely. Example: "Updates the version of Flask to v1.0.2"
  • In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.
• PR includes a summary of changes.
• PR adds unit tests, updates existing unit tests, OR documents why no test additions or modifications are needed.
• In case of new functionality, my PR adds documentation that describes how to use it.
  • All the public functions and the classes in the PR contain docstrings that explain what it does

[boring-cyborg]

Congratulations on your first Pull Request and welcome to Amundsen community! If you have any issues or are unsure about any anything please check our Contribution Guide (https://github.com/amundsen-io/amundsen/blob/main/CONTRIBUTING.md)

[chonyy]

Hi @feng-tao @mgorsk1 @dkunitsk , this is a unit test import error that does not relate to my change. This is also seen in other PRs, like here, here and here.
Is it possible for us to get past it or get our PR merged first?

[feng-tao]

@chonyy could you update the pr with rebase and fix the ci ? will take a look once it is updated

[chonyy]

@feng-tao Thanks! Will be working on passing the unit tests 😄

[feng-tao]

ping

[chonyy]

Hi @feng-tao , all ci unit tests are passed. Pls help me take a look at it when you have time, thanks 🙏

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.