Tomme / dbt-athena

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

Support AWS profile when accessing Glue, S3 #56

Open chronitis opened 2 years ago

chronitis commented 2 years ago

This is a (minimal) fix for #55 - direct construction of boto3.{Client,Resource} is replaced by constructing a boto3.session.Session (passing both region_name and aws_profile_name from the credentials, so they'll be used if non-null) and then using that to access resources.

chronitis commented 2 years ago

Rebased on 1.0.1

grufino commented 1 year ago

hey @chronitis only saw this now, I think we are solving the same issue but on two different places in slightly different ways: https://github.com/Tomme/dbt-athena/pull/124, we should probably stick to the same solution and use on both places (probably on others that call a new client as well)