-
Notifications
You must be signed in to change notification settings - Fork 14.1k
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
ImportError: No module named 'flask.exthook' #5006
Comments
Dup of #4953, please close |
Indeed, it mentions the same issue but the discussion says the exthook ImportError is fixed in superset v0.25.1 by forcing flask<1.0 (#4953 (comment)) but actually, in superset v0.25.1, the setup forces flask<=1.0, hence the version 1.0 in installed. |
You should probably open a PR fixing the issue :) |
Ooops, my bad, fixing in 0.25.4 (pushing this out now) |
Thanks !! |
Hello !
I ran into an error while running
superset db upgrade
:For what I can see, superset uses (or can use) flask==1.0 (
https://github.com/apache/incubator-superset/blob/master/setup.py#L58
) that no longer has aexthook
module (https://github.com/pallets/flask/releases/tag/1.0).This causes a ImportError on superset here : https://github.com/apache/incubator-superset/blob/master/superset/bin/superset#L9
When I install flask==0.12.2 before I install superset I no longer have the Error.
Maybe
flask<1.0.0
would work better? Or do you think I may be doing something wrong ?Make sure these boxes are checked before submitting your issue - thank you!
Superset version
0.25.2
Expected results
There should be no error
Actual results
ImportError is raised
Steps to reproduce
pip install superset==0.25.2
(by the way, at this point we get a warning
flask-appbuilder 1.10.0 has requirement Flask<0.12.99,>=0.12.1, but you'll have flask 1.0 which is incompatible.
)pip freeze
The text was updated successfully, but these errors were encountered: