Skip to content

Supportability - Azure Synapse Serverless SQL Pool#7589

Merged
jborsecnik merged 4 commits intoMicrosoftDocs:livefrom
charithcaldera:patch-5
May 25, 2022
Merged

Supportability - Azure Synapse Serverless SQL Pool#7589
jborsecnik merged 4 commits intoMicrosoftDocs:livefrom
charithcaldera:patch-5

Conversation

@charithcaldera
Copy link
Copy Markdown
Contributor

GRANT Database Scoped Credential Permissions works on Azure Synapse Analytics Serverless SQL Pool.
This is also required when a SQL user needs to access the data in external storage account.

Example :

CREATE LOGIN Mary WITH PASSWORD = '**********';
CREATE USER Mary FROM LOGIN Mary;
ALTER ROLE db_datareader ADD MEMBER Mary;

--CRAETE DB SCOPED CREDENTIALS
CREATE DATABASE SCOPED CREDENTIAL WorkspaceIdentity
WITH IDENTITY = 'Managed Identity';
GO

GRANT REFERENCES ON DATABASE SCOPED CREDENTIAL::[WorkspaceIdentity] TO [Mary];

GRANT Database Scoped Credential Permissions works on Azure Synapse Analytics Serverless SQL Pool.
This is also required when a SQL user needs to access the data in external storage account.
@PRMerger16
Copy link
Copy Markdown
Contributor

@charithcaldera : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@jborsecnik
Copy link
Copy Markdown
Contributor

@VanMSFT, There is no specific change provided here. Should we close this PR? Thanks.

Thanks.

@charithcaldera
Copy link
Copy Markdown
Contributor Author

@jborsecnik
The suggestion is to make this document available for Azure Synapse Serverless SQL Pool.
I have been communicating with one of the PMs for Serverless internally and this PR is created based on that.

@VanMSFT
Copy link
Copy Markdown
Member

VanMSFT commented May 19, 2022

@charithcaldera - Thanks! Can you clarify if it only applies to SQL Pools, or all of Azure Synapse?

@WilliamDAssafMSFT to confirm and make changes to the applies to when he gets back.

#reassign:WilliamDAssafMSFT

@azaricstefan
Copy link
Copy Markdown
Contributor

The suggestion is to make this document available for Azure Synapse Serverless SQL Pool.

I don't see the change in the moniker on top of the page? Only a space? Did you forgot to commit?

@PRMerger-2
Copy link
Copy Markdown
Contributor

@charithcaldera : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@charithcaldera
Copy link
Copy Markdown
Contributor Author

@charithcaldera - Thanks! Can you clarify if it only applies to SQL Pools, or all of Azure Synapse?

@WilliamDAssafMSFT to confirm and make changes to the applies to when he gets back.

#reassign:WilliamDAssafMSFT

Azure Synapse Analytics has both Dedicated SQL Pool and Serverless SQL pool.
This syntax works for Serverless SQL pool and I need to verify internally if this is supported in Dedicated SQL pool.

If this not supported to Dedicated SQL Pool in Synapse Analytics,
I think we will need to add a note that this is only supported for Serverless SQL Pool.

@charithcaldera
Copy link
Copy Markdown
Contributor Author

The suggestion is to make this document available for Azure Synapse Serverless SQL Pool.

I don't see the change in the moniker on top of the page? Only a space? Did you forgot to commit?

Sorry i was only made the suggestion and i just made the changes in to "Applies to" section.

Verified the syntax on Dedicated SQL Pools in Synapse.
Syntax fails in Dedicated SQL Pool and only the Serverless SQL Pools works for this syntax in Synapse. 

Since the document is marked as supported for "Azure Synapse Analytics", we have to specifically mention which part is supported. 
Modified the page with "GRANT Database Scoped Credential Permissions is only supported for serverless SQL pools in Azure Synapse Analytics."
@PRMerger4
Copy link
Copy Markdown
Contributor

@charithcaldera : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@charithcaldera
Copy link
Copy Markdown
Contributor Author

@VanMSFT Any update ?

@WilliamDAssafMSFT
Copy link
Copy Markdown
Contributor

@charithcaldera I am back in the office and can help here. I see in one of your above comments "I need to verify internally if this is supported in Dedicated SQL pool." Is this confirmed?

@charithcaldera
Copy link
Copy Markdown
Contributor Author

charithcaldera commented May 25, 2022

@charithcaldera I am back in the office and can help here. I see in one of your above comments "I need to verify internally if this is supported in Dedicated SQL pool." Is this confirmed?

Hello @WilliamDAssafMSFT
I tested this in both Dedicated SQL pool and Serverless SQL pool, but in Dedicated SQL pool, we are getting a error and same works in Serverless SQL Pool.

As a result of that, i have upadted the document with below additional comment

"GRANT Database Scoped Credential Permissions is only supported for serverless SQL pools in Azure Synapse Analytics."

GRANT REFERANCES ON DATABASES - SERVERLESS AND DEDICATED POOLS

Updating for latest style, update moniker and Applies to includes.
@PRMerger10
Copy link
Copy Markdown
Contributor

@WilliamDAssafMSFT : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@WilliamDAssafMSFT
Copy link
Copy Markdown
Contributor

@charithcaldera I've made some additional changes necessary to bring the article up to date for the moniker and applies to tags.
#sign-off

@PRMerger9
Copy link
Copy Markdown
Contributor

@WilliamDAssafMSFT: I'm sorry - only the author of this article, @VanMSFT, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

@charithcaldera
Copy link
Copy Markdown
Contributor Author

@charithcaldera I've made some additional changes necessary to bring the article up to date for the moniker and applies to tags. #sign-off

Hello @WilliamDAssafMSFT ,
Thank you,

I hope we can publish this soon.
please let me know if anything required from my end.

@jborsecnik jborsecnik merged commit 2643400 into MicrosoftDocs:live May 25, 2022
@WilliamDAssafMSFT
Copy link
Copy Markdown
Contributor

@charithcaldera all set. Thanks again for making taking the time to create this PR. Should be live later today.

@charithcaldera
Copy link
Copy Markdown
Contributor Author

@charithcaldera all set. Thanks again for making taking the time to create this PR. Should be live later today.

Thank you @WilliamDAssafMSFT & @VanMSFT for all the help

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.