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.

Sign up for GitHub

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

Jump to bottom

Add winkerberos support #140

Open
RiviqVH opened this issue Jan 5, 2023 · 2 comments
Open

Add winkerberos support #140

RiviqVH opened this issue Jan 5, 2023 · 2 comments
Labels
enhancement New feature or request

Comments

@RiviqVH
Copy link
Contributor

RiviqVH commented Jan 5, 2023

I am unable to install dbt-impala on a Windows machine from pip, because of the dependency on the kerberos package, which cannot be installed on Windows.

In order to support dbt-impala on Windows, it would be nice to have the winkerberos package instead (see this thread).
@RiviqVH RiviqVH mentioned this issue Jan 5, 2023
Merged
3 tasks
@RiviqVH
Copy link
Contributor Author

RiviqVH commented Jan 5, 2023

Note that my PR (#141) solves this issue by simply removing the kerberos requirement from the setup, but does not implement winkerberos in any way.

@niteshy niteshy added the enhancement New feature or request label Apr 24, 2023
@csringhofer
Copy link

Note that there is a PR in impyla about using winkerberos:
cloudera/impyla#504

I was hesitant to merge it as I was unable to test it, but it seems safe to fallback to winkerberos if kerberos is not found. Doing this in impyla should allow dbt-impala to work with winkerberos.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@niteshy @csringhofer @RiviqVH