[issue19299] test_asyncio refleak failures

Antoine Pitrou added the comment: Ok, should be fixed now! -- resolution: -> fixed stage: patch review -> committed/rejected status: open -> closed ___ Python tracker ___ __

[issue19299] test_asyncio refleak failures

Roundup Robot added the comment: New changeset 4019694a1ce4 by Antoine Pitrou in branch 'default': Issue #19299: fix refleak test failures in test_asyncio http://hg.python.org/cpython/rev/4019694a1ce4 -- nosy: +python-dev ___ Python tracker

[issue19299] test_asyncio refleak failures

Guido van Rossum added the comment: Looks good. How about adding executor.shutdown(wait=False)? E.g. def close(self): self._ready.clear() self._scheduled.clear() executor = self._default_executor if executor is not None: self._default_executor = No

[issue19299] test_asyncio refleak failures

Antoine Pitrou added the comment: (by the way, the part that really made a difference was losing the reference to the executor. I added the rest out of completeness.) -- ___ Python tracker

[issue19299] test_asyncio refleak failures

Antoine Pitrou added the comment: It seems the attached patch solves it. -- assignee: -> gvanrossum keywords: +patch stage: needs patch -> patch review Added file: http://bugs.python.org/file32231/base_event_close.patch ___ Python tracker

[issue19299] test_asyncio refleak failures

New submission from Antoine Pitrou: Either the tests forget to cleanup resources at the end, or the library itself has resource management issues. $ ./python -Wi -m test -R 3:3 test_asyncio [1/1] test_asyncio beginning 6 repetitions 123456 .. test_asyncio leaked [240, -677, -158] references

[issue19299] test_asyncio refleak failures

Changes by Antoine Pitrou : -- nosy: +larry ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python