Skip to content
This repository has been archived by the owner on Apr 8, 2024. It is now read-only.

feat(dbt-adapter): dbt-athena-community support #741

Merged
merged 4 commits into from
Feb 6, 2023

Conversation

mederka
Copy link
Collaborator

@mederka mederka commented Feb 3, 2023

dbt-athena from PyPI doesn't support dbt 1.3, but dbt-athena-community does

dbt-athena-community does not support dbt 1.4, so this branch doesn't need to be merged to main. Instead, we will release a patch to dbt-fal 1.3 in order to add support to dbt-athena-community.

Once either of these add support for dbt 1.4, we can create a new PR that incorporates those changes.

dbt-athena from PyPI doesn't support dbt 1.3, but dbt-athena-community does
https://github.com/dbt-athena/dbt-athena

dbt-athena-community does not support dbt 1.4, so this branch doesn't need to
be merged to main. Instead, we will release a patch to dbt-fal 1.3 in order to
add support to dbt-athena-community
@mederka mederka requested a review from chamini2 February 3, 2023 11:37
@mederka mederka changed the base branch from main to release-adapter-1.3 February 3, 2023 15:11
@mederka mederka marked this pull request as ready for review February 6, 2023 09:02
@mederka mederka requested a review from chamini2 February 6, 2023 09:02
Copy link
Member

@chamini2 chamini2 left a comment

Choose a reason for hiding this comment

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

Great! Should we add athena integration tests?

@mederka mederka merged commit f533e92 into release-adapter-1.3 Feb 6, 2023
chamini2 added a commit that referenced this pull request Feb 9, 2023
* tests(dbt-fal): add athena profile

* feat: add dbt-athena-community support

dbt-athena from PyPI doesn't support dbt 1.3, but dbt-athena-community does
https://github.com/dbt-athena/dbt-athena

dbt-athena-community does not support dbt 1.4, so this branch doesn't need to
be merged to main. Instead, we will release a patch to dbt-fal 1.3 in order to
add support to dbt-athena-community

* PR comments

* refactor: move some hacks location (#742)

---------

Co-authored-by: Matteo Ferrando <[email protected]>
chamini2 added a commit that referenced this pull request Feb 10, 2023
* tests(dbt-fal): add athena profile

* feat: add dbt-athena-community support

dbt-athena from PyPI doesn't support dbt 1.3, but dbt-athena-community does
https://github.com/dbt-athena/dbt-athena

dbt-athena-community does not support dbt 1.4, so this branch doesn't need to
be merged to main. Instead, we will release a patch to dbt-fal 1.3 in order to
add support to dbt-athena-community

* PR comments

* refactor: move some hacks location (#742)

---------

Co-authored-by: Matteo Ferrando <[email protected]>
chamini2 added a commit that referenced this pull request Feb 10, 2023
* tests(dbt-fal): add athena profile

* feat: add dbt-athena-community support

dbt-athena from PyPI doesn't support dbt 1.3, but dbt-athena-community does
https://github.com/dbt-athena/dbt-athena

dbt-athena-community does not support dbt 1.4, so this branch doesn't need to
be merged to main. Instead, we will release a patch to dbt-fal 1.3 in order to
add support to dbt-athena-community

* PR comments

* refactor: move some hacks location (#742)

---------

Co-authored-by: Matteo Ferrando <[email protected]>
mederka added a commit that referenced this pull request Apr 4, 2023
* tests(dbt-fal): add athena profile

* feat: add dbt-athena-community support

dbt-athena from PyPI doesn't support dbt 1.3, but dbt-athena-community does
https://github.com/dbt-athena/dbt-athena

dbt-athena-community does not support dbt 1.4, so this branch doesn't need to
be merged to main. Instead, we will release a patch to dbt-fal 1.3 in order to
add support to dbt-athena-community

* PR comments

* refactor: move some hacks location (#742)

---------

Co-authored-by: Matteo Ferrando <[email protected]>
mederka added a commit that referenced this pull request Apr 4, 2023
* feat(dbt-adapter): dbt-athena-community support (#741)

* tests(dbt-fal): add athena profile

* feat: add dbt-athena-community support

dbt-athena from PyPI doesn't support dbt 1.3, but dbt-athena-community does
https://github.com/dbt-athena/dbt-athena

dbt-athena-community does not support dbt 1.4, so this branch doesn't need to
be merged to main. Instead, we will release a patch to dbt-fal 1.3 in order to
add support to dbt-athena-community

* PR comments

* refactor: move some hacks location (#742)

---------

Co-authored-by: Matteo Ferrando <[email protected]>

* remove 1.3-specific hack

* Add tests

* comments

* Fix test setup

* update athena profile

* fix credential parsing in isolated envs

---------

Co-authored-by: Matteo Ferrando <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants