Tomme / dbt-athena

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

#56 #121 #124: Initializes boto3 session globally to support configured AWS profile when calling boto3 #125

Open w0rks74710n opened 1 year ago

w0rks74710n commented 1 year ago

This PR is yet another duplicate of #56 #121 #124. Due to the fact that #56 has been lying around for quite some time (outdated version of the code), #124 addresses the issue in only one instance of the adapter i.e. clean_up_table, I decided to go with the approach introduced in #121 by @cstruct and included a global variable that holds the boto3_session that can be retrieve at any point in time via get_boto3_session(). I would be happy to decline this PR in favour of #121 if preferred as it addresses the the issue at hand.

Thank you in advance.