Skip to content

Add TIMEZONE='UTC' to Snowflake Setup Guide #6351

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 4, 2024

Conversation

cjo2
Copy link
Member

@cjo2 cjo2 commented Apr 4, 2024

Proposed changes

One of the issues we faced in Linked Audiences were that TIMESTAMP_NTZ columns were being interpreted by the user/session's local time zone. By default, this is America/Los_Angeles. We are assuming this is meant to be a UTC field.

Merge timing

This can be merged as soon as this PR is approved.

Related issues (optional)

@cjo2 cjo2 force-pushed the update-snowflake-user-timezone branch from d0f74b9 to 953f7ba Compare April 4, 2024 17:14
@cjo2 cjo2 marked this pull request as ready for review April 4, 2024 17:18
@cjo2 cjo2 requested a review from pwseg as a code owner April 4, 2024 17:18
@pwseg pwseg added the content-update updates to content that are not new features, includes grammar fixes, added notes label Apr 4, 2024
@pwseg pwseg changed the title CLAS-194 - add TIMEZONE='UTC' to force queries using TIMESTAMP_NTZ to be considered UTC Add TIMEZONE='UTC' Snowflake Setup Guide Apr 4, 2024
@pwseg pwseg changed the title Add TIMEZONE='UTC' Snowflake Setup Guide Add TIMEZONE='UTC' to Snowflake Setup Guide Apr 4, 2024
@pwseg pwseg merged commit f076f5b into develop Apr 4, 2024
@pwseg pwseg deleted the update-snowflake-user-timezone branch April 4, 2024 17:23
Copy link
Contributor

github-actions bot commented Apr 4, 2024

Thank you for your contribution! Your pull request is merged, but may take a day or two to appear on the site.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content-update updates to content that are not new features, includes grammar fixes, added notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants