Skip to content

Conversation

@netonjm
Copy link
Member

@netonjm netonjm commented Nov 10, 2023

No description provided.

Copy link
Contributor

@anjagruenheid anjagruenheid left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jcamachor: Do you have any ideas on how we can add a test for this PR?

@@ -0,0 +1,37 @@
CREATE TABLE catalog_sales
(
CS_SOLD_DATE_SK int,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

indentation?

WITH (
FILE_TYPE = 'Csv',
CREDENTIAL = (IDENTITY = 'Shared Access Signature',
SECRET='${external_catalog_secret_token}'));
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should add this to the documentation somewhere.

@jcamachor jcamachor force-pushed the main branch 3 times, most recently from 4def705 to 7876be3 Compare July 10, 2024 15:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants