Skip to content
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

asyncio: Document that TCP_NODELAY is now used by default #72276

Closed
1st1 opened this issue Sep 12, 2016 · 14 comments
Closed

asyncio: Document that TCP_NODELAY is now used by default #72276

1st1 opened this issue Sep 12, 2016 · 14 comments
Labels
3.7 (EOL) end of life docs Documentation in the Doc dir easy topic-asyncio type-feature A feature request or enhancement

Comments

@1st1
Copy link
Member

1st1 commented Sep 12, 2016

BPO 28089
Nosy @gvanrossum, @vstinner, @ned-deily, @berkerpeksag, @1st1, @Mariatta
Files
  • issue28089.patch
  • issue28089v2.patch
  • Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.

    Show more details

    GitHub fields:

    assignee = None
    closed_at = <Date 2016-12-12.21:46:32.516>
    created_at = <Date 2016-09-12.01:33:48.169>
    labels = ['easy', 'type-feature', '3.7', 'docs', 'expert-asyncio']
    title = 'asyncio: Document that TCP_NODELAY is now used by default'
    updated_at = <Date 2016-12-16.07:54:15.332>
    user = 'https://github.com/1st1'

    bugs.python.org fields:

    activity = <Date 2016-12-16.07:54:15.332>
    actor = 'ned.deily'
    assignee = 'docs@python'
    closed = True
    closed_date = <Date 2016-12-12.21:46:32.516>
    closer = 'yselivanov'
    components = ['Documentation', 'asyncio']
    creation = <Date 2016-09-12.01:33:48.169>
    creator = 'yselivanov'
    dependencies = []
    files = ['45770', '45772']
    hgrepos = []
    issue_num = 28089
    keywords = ['patch', 'easy']
    message_count = 14.0
    messages = ['282459', '282466', '282470', '282471', '282495', '282497', '282498', '282510', '282511', '283050', '283053', '283054', '283377', '283384']
    nosy_count = 8.0
    nosy_names = ['gvanrossum', 'vstinner', 'ned.deily', 'docs@python', 'python-dev', 'berker.peksag', 'yselivanov', 'Mariatta']
    pr_nums = []
    priority = None
    resolution = 'fixed'
    stage = 'resolved'
    status = 'closed'
    superseder = None
    type = 'enhancement'
    url = 'https://bugs.python.org/issue28089'
    versions = ['Python 3.6', 'Python 3.7']

    @1st1 1st1 added docs Documentation in the Doc dir topic-asyncio easy type-feature A feature request or enhancement labels Sep 12, 2016
    @ned-deily
    Copy link
    Member

    This is marked as a release blocker but, since it's just a doc change, I'm not going to hold 3.6.0 for it. It would be nice to get it in, though.

    @vstinner
    Copy link
    Member

    vstinner commented Dec 5, 2016

    Yury: I don't understand your issue, can you please elaborate? Do you mean that the default value of the TCP_NODELAY changed in Python 3.6? Otherwise, why do you consider it as a release blocker?

    The Python 3.6 release must be blocked by a TCP flag? Really?

    @Mariatta
    Copy link
    Member

    Mariatta commented Dec 5, 2016

    The change is that TCP_NODELAY option is set by default in 3.6. It was not the case in 3.5.

    @vstinner vstinner changed the title Document TCP_NODELAY by default asyncio: Document that TCP_NODELAY is now used by default Dec 5, 2016
    @vstinner
    Copy link
    Member

    vstinner commented Dec 5, 2016

    The change is that TCP_NODELAY option is set by default in 3.6. It was not the case in 3.5.

    Ah, it's a change in _asyncio_, ok. I missed that from the issue title, so I changed the title.

    @Mariatta
    Copy link
    Member

    Mariatta commented Dec 6, 2016

    I added the following to Doc/library/asyncio-protocol.rst

    .. versionchanged:: 3.6.0
    The socket option TCP_NODELAY is now set by default.

    Let me know if this patch works.

    @berkerpeksag
    Copy link
    Member

    +.. versionchanged:: 3.6.0

    3.6.0 -> 3.6

    + The socket option TCP_NODELAY is now set by default.

    TCP_NODELAY -> TCP_NODELAY

    @Mariatta
    Copy link
    Member

    Mariatta commented Dec 6, 2016

    Thanks, Berker :)
    Updated.

    @python-dev
    Copy link
    Mannequin

    python-dev mannequin commented Dec 6, 2016

    New changeset 726308cfe3b5 by Victor Stinner in branch '3.6':
    catch_warnings() calls showwarning() if overriden
    https://hg.python.org/cpython/rev/726308cfe3b5

    @python-dev
    Copy link
    Mannequin

    python-dev mannequin commented Dec 6, 2016

    New changeset 150d36dbe3ba by Victor Stinner in branch '3.6':
    warnings: Fix the issue number
    https://hg.python.org/cpython/rev/150d36dbe3ba

    @ned-deily
    Copy link
    Member

    Yury, look good to you?

    @ned-deily ned-deily added the 3.7 (EOL) end of life label Dec 12, 2016
    @python-dev
    Copy link
    Mannequin

    python-dev mannequin commented Dec 12, 2016

    New changeset 853e3f4d6cd9 by Yury Selivanov in branch '3.6':
    Issue bpo-28089: Document TCP_NODELAY in asyncio
    https://hg.python.org/cpython/rev/853e3f4d6cd9

    New changeset 0d209cc7ffdc by Yury Selivanov in branch 'default':
    Merge 3.6 (issue bpo-28089)
    https://hg.python.org/cpython/rev/0d209cc7ffdc

    @1st1
    Copy link
    Member Author

    1st1 commented Dec 12, 2016

    Yury, look good to you?

    Yes; committed the patch with a small addition. Thanks, Mariatta!

    @1st1 1st1 closed this as completed Dec 12, 2016
    @python-dev
    Copy link
    Mannequin

    python-dev mannequin commented Dec 16, 2016

    New changeset dfd1019f75f9 by Yury Selivanov in branch '3.6':
    Issue bpo-28089: Document TCP_NODELAY in asyncio
    https://hg.python.org/cpython/rev/dfd1019f75f9

    @ned-deily
    Copy link
    Member

    [cherrypicked for 3.6.0rc2]

    @ezio-melotti ezio-melotti transferred this issue from another repository Apr 10, 2022
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Labels
    3.7 (EOL) end of life docs Documentation in the Doc dir easy topic-asyncio type-feature A feature request or enhancement
    Projects
    None yet
    Development

    No branches or pull requests

    5 participants