Requests do not share HTTP(S) connection pool

[azure]

Requests do not share HTTP(S) connection pool

[akx]

Ah, apparently

client = ComputeManagementClient(...)
client.config.keep_alive = True

makes the client instance not close the underlying session. Whether or that's safe, I have no idea...

[lmazuel]

Hi @akx
Currently, you have two ways to achieve this:

• Use the client as a context manager (with ComputeManagementClient(**parameters) as client)
• Use the flag you found (and the close() method when done)

It was not activated in early SDK, and I didn't activated it by default with current SDK, because that's a breaking change. This code will leak socket connection for example:

for el in myloop():
    client = ComputeClientManagement(**parameters)
    client.virtual_machines.list(**el)

Next major version of client will introduce async, and I decided to make the pool enabled by default with a disclaimer in the ChangeLog.

Hope this clarify :). Thanks!

https://github.com/Azure/msrest-for-python#2018-04-18-version-0428

[lmazuel]

Was commited part of Autorest for Python 4.0.63 as the default behavior: enabled pool per client. This will be shipped at the same time as async support in the next couples of month. Being, if the SDK supports async, it means the connection pool is activated by default.