Support quoted table names

[tomfroehle]

Duplicates #2473 because I cannot contribute with my enterprise account anymore.

Original PR description:

Why make this change?

• Fixes [Bug]: Quoted Table name are not supported #2385

What is this change?

• In SqlMetadataProvider.PopulateColumnDefinitionsWithReadOnlyFlag the QuoteIdentifier-method of the SqlQueryBuilder was not used to add quotes to the table name

How was this tested?

• Integration Tests -> I'm happy to extend the integration tests, if this fix seems like a good idea
• Unit Tests
• manually

Sample Request(s)

Sample configuraiton:

"MyTable": {
  "source": {
    "object": "schema.\"MyTable\"",
    "type": "table"
  }
}

[tomfroehle]

@microsoft-github-policy-service agree

[aaronburtle]

Thanks for following up!

[JerryNixon]

What happens to existing customers who have indicated their table name but have used the incorrect casing for those tables? Will this break those customers?

[Aniruddh25]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[Aniruddh25]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 6 pipeline(s).

[Aniruddh25]

What happens to existing customers who have indicated their table name but have used the incorrect casing for those tables? Will this break those customers?

Good question. Lets consider the 2 cases if customers have used incorrect casing,

1. and their table names are case sensitive - it must not have been working for them anyway right now without the quotes. After adding the quotes, it will still not work since they are doing incorrect casing.

2. and their table names are case insensitive - adding quotes shouldn't be a problem, since the names are case insensitive. Should continue to work. @aaronburtle, could you please confirm this scenario?

[Aniruddh25]

Approving existing changes. Any leftover testing should follow up in the subsequent issue tracked by @aaronburtle