-
-
Notifications
You must be signed in to change notification settings - Fork 32.1k
[2.7] bpo-32362: Fix references to non-existent multiprocessing.Connection() (GH-6223) #6646
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…ction() (pythonGH-6223). (cherry picked from commit 9f3535c) Co-authored-by: Bo Bayles <[email protected]>
Thank you for your backport @bbayles. But things are more complex in 2.7. |
I think I've done that by un-setting the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't know what is the bests solution here. Removing the multiprocessing.
looks like an improvement.
Doc/library/multiprocessing.rst
Outdated
Returns a pair ``(conn1, conn2)`` of :class:`Connection` objects representing | ||
the ends of a pipe. | ||
Returns a pair ``(conn1, conn2)`` of :class:`Connection` objects | ||
representing the ends of a pipe. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unneeded change.
Doc/library/multiprocessing.rst
Outdated
@@ -1943,7 +1948,7 @@ Listeners and Clients | |||
:synopsis: API for dealing with sockets. | |||
|
|||
Usually message passing between processes is done using queues or by using | |||
:class:`~multiprocessing.Connection` objects returned by | |||
:class:`~Connection` objects returned by |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
~
is not needed.
Doc/library/multiprocessing.rst
Outdated
@@ -2139,10 +2144,10 @@ an ``'AF_PIPE'`` address rather than an ``'AF_UNIX'`` address. | |||
Authentication keys | |||
~~~~~~~~~~~~~~~~~~~ | |||
|
|||
When one uses :meth:`Connection.recv <multiprocessing.Connection.recv>`, the | |||
When one uses :meth:`Connection.recv <Connection.recv>`, the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just :meth:`Connection.recv`
.
Doc/library/multiprocessing.rst
Outdated
data received is automatically | ||
unpickled. Unfortunately unpickling data from an untrusted source is a security | ||
risk. Therefore :class:`Listener` and :func:`Client` use the :mod:`hmac` module | ||
unpickled. Unfortunately unpickling data from an untrusted source is a security |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unneeded (and unwanted) changes.
Connection objects allow the sending and receiving of picklable objects or | ||
strings. They can be thought of as message oriented connected sockets. | ||
|
||
Connection objects are usually created using :func:`Pipe` -- see also | ||
Connection objects are usually created using | ||
:func:`Pipe <multiprocessing.Pipe>` -- see also | ||
:ref:`multiprocessing-listeners-clients`. | ||
|
||
.. class:: Connection |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe add the :noindex:
option?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh, I wanted just exclude it from the index, but seems this doesn't work this way.
OK, thanks. I've changed those things. |
Er, it seems I pushed the wrong thing - apologies. |
Connection objects allow the sending and receiving of picklable objects or | ||
strings. They can be thought of as message oriented connected sockets. | ||
|
||
Connection objects are usually created using :func:`Pipe` -- see also | ||
Connection objects are usually created using | ||
:func:`Pipe <multiprocessing.Pipe>` -- see also | ||
:ref:`multiprocessing-listeners-clients`. | ||
|
||
.. class:: Connection |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh, I wanted just exclude it from the index, but seems this doesn't work this way.
This PR is a backport of #6223 to 2.7.
While fixing the conflict, I noticed a mistake in the 2.7 docs. This section states there are "two" exceptions, but then goes on to define only one. I will make a separate PR for that.
CC: @serhiy-storchaka
https://bugs.python.org/issue32362