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

ci: temporarily add pure-sasl to impala conda dependencies #2987

Merged
merged 1 commit into from
Sep 18, 2021

Conversation

cpcloud
Copy link
Member

@cpcloud cpcloud commented Sep 18, 2021

This PR adds pure-sasl to the impala backend dependencies due to impyla
not declaring a hard dependency on pure-sasl (cloudera/impyla#471).

@github-actions
Copy link
Contributor

github-actions bot commented Sep 18, 2021

Unit Test Results

       19 files         19 suites   1h 42m 22s ⏱️
10 756 tests   8 402 ✔️   2 354 💤 0 ❌
54 461 runs  41 187 ✔️ 13 274 💤 0 ❌

Results for commit 9e1bbaa.

♻️ This comment has been updated with latest results.

@cpcloud cpcloud added this to the Next release milestone Sep 18, 2021
@cpcloud
Copy link
Member Author

cpcloud commented Sep 18, 2021

In the interest of getting our CI back to passing, I'm going to merge this on green.

@cpcloud cpcloud added bug Incorrect behavior inside of ibis ci Continuous Integration issues or PRs dependencies Issues or PRs related to dependencies labels Sep 18, 2021
@jreback
Copy link
Contributor

jreback commented Sep 18, 2021

sgtm

@cpcloud cpcloud merged commit 9e1bbaa into ibis-project:master Sep 18, 2021
@cpcloud cpcloud deleted the temp-add-pure-sasl branch September 18, 2021 14:21
@cpcloud cpcloud removed this from the Next release milestone Jan 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Incorrect behavior inside of ibis ci Continuous Integration issues or PRs dependencies Issues or PRs related to dependencies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants