Tomme / dbt-athena

The athena adapter plugin for dbt (https://getdbt.com)
Apache License 2.0
142 stars 79 forks source link

Implementing snapshots #19

Closed mrshu closed 2 years ago

mrshu commented 3 years ago

Hey @Tomme ,

First of all, thanks a ton for dbt-athena -- it works very nicely!

I was wondering would would it take to implement snapshots with Athena. Since it doesn't really do updates, it will be quite a challenge but from the outset it does not seem completely impossible.

Would you see it that way as well? It seems the Presto adapter is also not implementing it, so I was wondering what would your thoughts on something like this be.

Thanks!

Tomme commented 3 years ago

Hey @mrshu 👋

I would agree that it is not impossible however to implement it would mean allowing the adapter destructive actions on your data within S3 which one could argue is out of the scope for Athena (and thus this adapter). I might ask on the #db-athena Slack channel for people thoughts and opinions.

Regardless a rough approach to how we could implement snapshots and upsert incremental models could be:

Tomme commented 2 years ago

PR currently open to implement this feature: https://github.com/Tomme/dbt-athena/pull/27

mrshu commented 2 years ago

Very cool, thanks a ton @Tomme !