Skip to content

Conversation

@XYGUAN
Copy link
Contributor

@XYGUAN XYGUAN commented Aug 23, 2021

No description provided.

@XYGUAN XYGUAN requested a review from James-Mission August 23, 2021 16:04
materialized='incremental',
sort='block_timestamp',
unique_key='block_id',
unique_key='block_id || event_from || event_to',
Copy link
Contributor

Choose a reason for hiding this comment

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

this unique key is wrong there are duplicates on these 3 keys

select block_id,
event_from,
event_to,
count(1)
from terra.transfers
where block_timestamp::date = '2021-08-01'
group by 1,2,3
order by 4 desc

{{
config(
materialized='incremental',
sort='block_timestamp',
Copy link
Contributor

Choose a reason for hiding this comment

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

remove any sort keys for the configs they don't do anything on snowflake

Copy link
Contributor

@James-Mission James-Mission left a comment

Choose a reason for hiding this comment

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

see notes

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.

3 participants