search

Tomme / dbt-athena

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

Add thread lock to boto3 resource #76

Closed nicholsondarius closed 2 years ago

nicholsondarius commented 2 years ago

https://github.com/Tomme/dbt-athena/pull/66 was merged recently but I have still been getting the errors it set out to resolve. This documentation on the boto3 resource says that the resource is also not thread safe. This PR adds a thread lock to the instantiation of the boto3 resource.

nicholsondarius commented 2 years ago

Leaving this PR in draft until #74 is merged or closed as it mentions not needing to thread lock.

nicholsondarius commented 2 years ago

I was still getting errors because my GitHub Actions self-hosted runner wasn't updating to the latest version of the adapter. After force reinstalling the adapter, things have been running fine.