[GitHub] [incubator-superset] sturmer opened a new issue #7620: Make Requests library an optional dependency

[GitBox <gi...@apache.org>]

sturmer opened a new issue #7620: Make Requests library an optional dependency
URL: https://github.com/apache/incubator-superset/issues/7620
 
 
   The Requests library seems to be legally problematic so it was suggested to get rid of it as a dependency.
   
   Quoting @mistercrunch :
   
   > It appears it's used
   in its simplest form in a few places (requests.get) which should be easy to
   replace with urrlib2. The Druid connector uses a more advanced feature
   (basic auth), but that could be made just an optional dependency, as we're
   deprecating the Druid connector, to be replaced by the SQLAlchemy Druid
   connector/dialect. There it's just a matter of moving `pydruid` and
   `requests` to the "extra_requires" section of our setup.py, and doing late
   imports.
   
   CC: @villebro 

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org