-
Notifications
You must be signed in to change notification settings - Fork 200
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
Pydruid 0.5.9 installation fails on Python 3.7 (with pytest ??) #198
Comments
@potiuk I'm hitting the same error on Python 3.7.5, based on quick googling it seems to be a problem with the way |
Yep. We did that as a workaround we added limits for pydruid. Happy to remove it when it's fixed :) |
Also running into this. |
FWIW, pytest predates my involvement in this repo. Happy to review code that will make us 3.7.x compatible. |
Could this be fixed as part of #190 by removing |
Should we move away from pytest and onto something like nosetest? How much work would that represent? |
Agreed, we need to deprecate |
Cool. Will try it :) |
Thanks! Tested and removed the upper bound here apache/airflow#9965 . After we merge it to master it should automatically land in our "official" constraints file with 0.6.1 (current latest pydruid version) |
When we try to install released yesterday pydruid 0.5.9 on python 3.7 during our CI pipeline it fails to install. Looks like it is somewhow related to pytest runner (I guess some pytest dependency in druid moved from setup-requires to install-requires maybe?)
Example here: https://travis-ci.org/github/apache/airflow/jobs/662274100?utm_medium=notification&utm_source=github_status
The text was updated successfully, but these errors were encountered: