alluxio-py icon indicating copy to clipboard operation
alluxio-py copied to clipboard

Published 20 hours ago • verified publisher icon Alluxio

Alluxio client as context manager that closes session.

Open peterroelants opened this issue 4 years ago • 1 comments

Ideally, shouldn't the Alluxio Client be used as a context manager that opens the requests.Session() as a context manager and closes this session properly (closing all the connection adapters) when the Client exits?

Currently the session object is kept open.

peterroelants avatar Feb 18 '21 15:02 peterroelants

I'm happy to create and submit a PR if this would indeed be better practice.

peterroelants avatar Feb 18 '21 15:02 peterroelants