Skip to content

bpo-34323: IocpProactor: prevent modification if closed#11494

Merged
vstinner merged 1 commit into
python:masterfrom
vstinner:iocp_closed
Jan 10, 2019
Merged

bpo-34323: IocpProactor: prevent modification if closed#11494
vstinner merged 1 commit into
python:masterfrom
vstinner:iocp_closed

Conversation

@vstinner

@vstinner vstinner commented Jan 10, 2019

Copy link
Copy Markdown
Member
  • _wait_for_handle(), _register() and _unregister() methods of
    IocpProactor now raise an exception if closed
  • Add "closed" to IocpProactor.repr()
  • Simplify IocpProactor.close()

https://bugs.python.org/issue34323

* _wait_for_handle(), _register() and _unregister() methods of
  IocpProactor now raise an exception if closed
* Add "closed" to IocpProactor.__repr__()
* Simplify IocpProactor.close()
@vstinner vstinner changed the title IocpProactor: prevent modification if closed bpo-34323: IocpProactor: prevent modification if closed Jan 10, 2019
@vstinner

Copy link
Copy Markdown
Member Author

I wrote this change to prepare IocpProactor.close() fix for https://bugs.python.org/issue34323 and to make the code more reliable.

@asvetlov asvetlov left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@vstinner vstinner merged commit 9b07681 into python:master Jan 10, 2019
@vstinner vstinner deleted the iocp_closed branch January 10, 2019 10:23
vstinner added a commit that referenced this pull request Jan 15, 2019
* IocpProactor: prevent modification if closed (GH-11494)

* _wait_for_handle(), _register() and _unregister() methods of
  IocpProactor now raise an exception if closed
* Add "closed" to IocpProactor.__repr__()
* Simplify IocpProactor.close()

(cherry picked from commit 9b07681)

* bpo-34323: Enhance IocpProactor.close() log (GH-11555)

IocpProactor.close() now uses time to decide when to log: wait 1
second before the first log, then log every second. Log also the
number of seconds since close() was called.

(cherry picked from commit b1e4573)

* bpo-34323: Enhance IocpProactor.close() log again (GH-11563)

Add repr(self) to the log to display the number of pending overlapped
in the log.

(cherry picked from commit b91140f)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants