-
Notifications
You must be signed in to change notification settings - Fork 237
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
Use PyHive pure-sasl import #818
Merged
Merged
Changes from 9 commits
Commits
Show all changes
35 commits
Select commit
Hold shift + click to select a range
d359a8f
Use PyHive pure-sasl import
colin-rogers-dbt bb3c688
Test PyHive pure-sasl import
colin-rogers-dbt 44f1b14
update requirements.txt
colin-rogers-dbt 7360353
use github fix for >3.11
colin-rogers-dbt 3779c2d
use github fix for >3.11
colin-rogers-dbt 4982589
fix pyhive spec
colin-rogers-dbt 71030e3
remove unnecessary requests extra
colin-rogers-dbt ddd1f79
remove unnecessary sasl install
colin-rogers-dbt bcb47f5
add changie
colin-rogers-dbt b5b2b20
specify no-binary for pyodbc install
colin-rogers-dbt 152a2ee
conditionally import sasl vs puresasl
colin-rogers-dbt 0a51daa
conditionally import sasl vs pure-sasl
colin-rogers-dbt 3c18e5b
Merge branch 'main' into usePureSasl
colin-rogers-dbt c80e599
experiment with pure sasl SASLClient
colin-rogers-dbt 50a5a54
file formatting
colin-rogers-dbt 9e738b1
test using only puresasl
colin-rogers-dbt 52a2ba7
test using only puresasl
colin-rogers-dbt a03e364
update requirements.txt
colin-rogers-dbt 7987673
remove sasl from req
colin-rogers-dbt 3941979
remove unused sys dependency
colin-rogers-dbt 6ff75f7
Merge branch 'main' into usePureSasl
colin-rogers-dbt 078d2c6
Merge branch 'main' into usePureSasl
mikealfare 6d6c561
Merge branch 'main' into usePureSasl
mikealfare ef99d3e
test spark-session
colin-rogers-dbt 645c695
fix pure-sasl
colin-rogers-dbt 86d9767
allow pre-release installs
colin-rogers-dbt 5ea5c55
allow pre-release installs
colin-rogers-dbt 9ea4393
Merge branch 'main' into usePureSasl
colin-rogers-dbt 3486590
Merge branch 'main' into usePureSasl
colin-rogers-dbt cede12f
use pyhive 0.7.0
colin-rogers-dbt 215b9fa
merge main
colin-rogers-dbt c88567f
add pyhive back to reqs
colin-rogers-dbt 2851b78
fix setup.py and remove sasl from dev-requirements.txt
colin-rogers-dbt 9e0769e
Update Dependencies-20230628-121341.yaml
colin-rogers-dbt fc2c610
remove unnecessary pip_pre arg from tox.ini
colin-rogers-dbt File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,6 @@ | ||
kind: Dependencies | ||
body: Use PyHive pure-sasl import for >=3.11 | ||
time: 2023-06-28T12:13:41.141588-07:00 | ||
custom: | ||
Author: colin-rogers-dbt | ||
PR: "818" |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -30,5 +30,4 @@ wheel~=0.40 | |
|
||
# Adapter specific dependencies | ||
mock~=5.0 | ||
sasl~=0.3.1 | ||
thrift_sasl~=0.4.3 |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lol