Skip to content

Added an option to use reusable connections - #234

Closed
nnappe wants to merge 1 commit into
watson-developer-cloud:masterfrom
nnappe:master
Closed

Added an option to use reusable connections#234
nnappe wants to merge 1 commit into
watson-developer-cloud:masterfrom
nnappe:master

Conversation

@nnappe

@nnappe nnappe commented Jul 11, 2017

Copy link
Copy Markdown

This reuses HTTP connections so future REST calls skip the handshake. We've seen aprox 1s reductions in time to complete each requests (even from within IBM internal network). Might not be as much of a difference from within Bluemix.
The default behavior is not changed (unless objects are created with a use_connection_pool=True parameter). If so, a request.Session object is created at initialization time and used for future requests.

…r is not changed. It creates a request.Session object at initialization time and uses it for future requests.
@CLAassistant

CLAassistant commented Jul 11, 2017

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@codecov-io

codecov-io commented Jul 11, 2017

Copy link
Copy Markdown

Codecov Report

Merging #234 into master will decrease coverage by 2.83%.
The diff coverage is 90.9%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #234      +/-   ##
==========================================
- Coverage   84.31%   81.48%   -2.84%     
==========================================
  Files          24       24              
  Lines        1167     1172       +5     
==========================================
- Hits          984      955      -29     
- Misses        183      217      +34
Impacted Files Coverage Δ
watson_developer_cloud/tone_analyzer_v3.py 92.3% <ø> (-7.7%) ⬇️
watson_developer_cloud/tradeoff_analytics_v1.py 100% <ø> (ø) ⬆️
watson_developer_cloud/personality_insights_v3.py 93.33% <ø> (ø) ⬆️
watson_developer_cloud/discovery_v1.py 100% <ø> (ø) ⬆️
watson_developer_cloud/__init__.py 100% <ø> (ø) ⬆️
..._developer_cloud/natural_language_classifier_v1.py 100% <100%> (ø) ⬆️
watson_developer_cloud/retrieve_and_rank_v1.py 98.14% <100%> (ø) ⬆️
watson_developer_cloud/authorization_v1.py 62.5% <100%> (ø) ⬆️
...veloper_cloud/natural_language_understanding_v1.py 100% <100%> (ø) ⬆️
..._developer_cloud/watson_developer_cloud_service.py 63.95% <83.33%> (-8.96%) ⬇️
... and 5 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 08761ca...f86ed65. Read the comment docs.

@nnappe nnappe closed this Jul 11, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants