Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

dbt found two macros named "set_table_classification" in the project "dbt_athena". #103

Closed
mrshu opened this issue Dec 31, 2022 · 3 comments · Fixed by #106
Closed

dbt found two macros named "set_table_classification" in the project "dbt_athena". #103

mrshu opened this issue Dec 31, 2022 · 3 comments · Fixed by #106

Comments

@mrshu
Copy link

mrshu commented Dec 31, 2022

I ran into this when installing dbt-athena-community via pip install dbt-athena-community on dbt=1.3.1.

The issue was that the package dbt-athena (https://github.com/Tomme/dbt-athena) has already been installed on the target system, leading to the two macros of the same name being being defined.

The fix was to uninstall one of the dbt-athena* packages -- in my case pip uninstall dbt-athena.

@mrshu
Copy link
Author

mrshu commented Dec 31, 2022

I only opened this bug for the benefit of anyone who might run into this in the future. I don't think there is much the dbt-athena project can do to help with this issue, other than to make sure it can be looked up, which this issue should help with.

I am hence closing it.

@mrshu mrshu closed this as completed Dec 31, 2022
@nicor88
Copy link
Contributor

nicor88 commented Dec 31, 2022

@mrshu Thanks for this, actually might make sense to add this to the Readme? A small section that says that if dbt-athena-adapter is already installed, must be disinstalled to avoid conflicts.

@mrshu
Copy link
Author

mrshu commented Dec 31, 2022

@nicor88 sure, that would make sense.

Let me try to put together a quick PR 🙂

mrshu added a commit to mrshu/dbt-athena that referenced this issue Dec 31, 2022
* Add a quick note on avoiding the conflict with `dbt-athena-adapter`.

* Fixes dbt-labs#103

Signed-off-by: mrshu <[email protected]>
mrshu added a commit to mrshu/dbt-athena that referenced this issue Dec 31, 2022
* Add a quick note on avoiding the conflict with `dbt-athena-adapter`.

* Fixes dbt-labs#103

Signed-off-by: mrshu <[email protected]>
mrshu added a commit to mrshu/dbt-athena that referenced this issue Dec 31, 2022
* Add a quick note on avoiding the conflict with `dbt-athena-adapter`.

* Fixes dbt-labs#103

Signed-off-by: mrshu <[email protected]>
mrshu added a commit to mrshu/dbt-athena that referenced this issue Dec 31, 2022
* Add a quick note on avoiding the conflict with `dbt-athena-adapter`.

* Fixes dbt-labs#103

Signed-off-by: mrshu <[email protected]>
nicor88 pushed a commit that referenced this issue Jan 3, 2023
* Add a quick note on avoiding the conflict with `dbt-athena-adapter`.

* Fixes #103

Signed-off-by: mrshu <[email protected]>

Signed-off-by: mrshu <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants