You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Automatically created Loops should be closed automatically #14
Constructor of msgpackrpc.Session class has an optional argument loop=None. When loop is not specified, new Loop will be automatically created in the constructor.
However, as automatically-created Loops are not closed automatically, it uses more file descriptors than expected. This becomes an issue for clients that connects/disconnects frequently.
I think automatically-created Loops should be closed (i.e., session._loop._ioloop.close()) when its associated Session is closed.
I have two ideas:
If Loop instance is not specified by user (i.e., automatically created in Session constructor), close the Loop when session.close() method is called.
Add close_loop=True to the Session constructor argument. Close the Loop only if close_loop is set to True when session.close() method is called. This behavior is similar to closefd option of io.FileIO.
In addition, it is better to add close() method to msgpackrpc.Loop class, to allow users to close the internal tornado IOLoop.
Activity
wosiu commentedon Sep 9, 2018
+1
I use rpc client in my django server (to call c++ rpc server) and after a while I get:
[Errno 24] Too many open fileskmaehashi commentedon Sep 10, 2018
Current workaround: