Skip to content

Commit

Permalink
Migrate gevent's wsgi to pywsgi (allenai#1272)
Browse files Browse the repository at this point in the history
Since gevent 1.0a1 was released in 2011, wsgi is just an alias for pywsgi.
In version 1.3 wsgi is completely removed, causing allennlp to break.
For more info check  http://www.gevent.org/api/gevent.wsgi.html

Test plan:
run scripts/verify.py on trunk, all test pass.
  • Loading branch information
mantili authored and joelgrus committed May 24, 2018
1 parent 587f754 commit 92c2269
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion allennlp/service/server_flask.py
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@

from flask import Flask, request, Response, jsonify, send_file, send_from_directory
from flask_cors import CORS
from gevent.wsgi import WSGIServer
from gevent.pywsgi import WSGIServer

import psycopg2

Expand Down
2 changes: 1 addition & 1 deletion allennlp/service/server_simple.py
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@

from flask import Flask, request, Response, jsonify, send_file, send_from_directory
from flask_cors import CORS
from gevent.wsgi import WSGIServer
from gevent.pywsgi import WSGIServer

from allennlp.common import JsonDict
from allennlp.common.util import import_submodules
Expand Down

0 comments on commit 92c2269

Please sign in to comment.