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

[EASY] SubinterpThreadingTests.test_threads_join_2() of test_threading leaks references #74721

Closed
vstinner opened this issue Jun 1, 2017 · 4 comments
Labels
3.7 (EOL) end of life easy performance Performance or resource usage

Comments

@vstinner
Copy link
Member

vstinner commented Jun 1, 2017

BPO 30536
Nosy @vstinner, @ericsnowcurrently, @zware, @serhiy-storchaka, @sayanchowdhury
Superseder
  • bpo-30547: [Windows] SubinterpreterTest.test_callbacks_leak() of test_atexit leaks references
  • 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 2017-06-08.10:20:50.294>
    created_at = <Date 2017-06-01.10:42:19.613>
    labels = ['easy', '3.7', 'performance']
    title = '[EASY] SubinterpThreadingTests.test_threads_join_2() of test_threading leaks references'
    updated_at = <Date 2017-06-08.10:47:04.664>
    user = 'https://github.com/vstinner'

    bugs.python.org fields:

    activity = <Date 2017-06-08.10:47:04.664>
    actor = 'serhiy.storchaka'
    assignee = 'none'
    closed = True
    closed_date = <Date 2017-06-08.10:20:50.294>
    closer = 'vstinner'
    components = []
    creation = <Date 2017-06-01.10:42:19.613>
    creator = 'vstinner'
    dependencies = []
    files = []
    hgrepos = []
    issue_num = 30536
    keywords = ['easy (C)']
    message_count = 4.0
    messages = ['294932', '294997', '295024', '295393']
    nosy_count = 5.0
    nosy_names = ['vstinner', 'eric.snow', 'zach.ware', 'serhiy.storchaka', 'sayanchowdhury']
    pr_nums = []
    priority = 'normal'
    resolution = 'duplicate'
    stage = 'resolved'
    status = 'closed'
    superseder = '30547'
    type = 'resource usage'
    url = 'https://bugs.python.org/issue30536'
    versions = ['Python 3.7']

    @vstinner
    Copy link
    Member Author

    vstinner commented Jun 1, 2017

    Ah! I succeeded to hack a tool to bisect -m test -R 3:3 failure (issue bpo-29512, I will publish it later, right now it's too hackish), so I found a first guilty of failure on the Gentoo Refleak failure.

    Maybe it's related to recent Eric Snow's work. Maybe not. I didn't dig the bug yet.

    haypo@selma$ ./python -m test -R 3:3 -m test_threads_join_2 test_threading
    Run tests sequentially
    0:00:00 load avg: 0.16 [1/1] test_threading
    beginning 6 repetitions
    123456
    ......
    test_threading leaked [3, 3, 3] references, sum=9
    test_threading failed

    1 test failed:
    test_threading

    Total duration: 2 sec
    Tests result: FAILURE

    @vstinner vstinner added 3.7 (EOL) end of life performance Performance or resource usage labels Jun 1, 2017
    @vstinner
    Copy link
    Member Author

    vstinner commented Jun 2, 2017

    I tagged this issue as easy. I consider that it's a good exercice for new contributors. Core developers: please let new contributors try to fix it since this issue is not critical, try to explain how to fix it rather than writing the fix ;-)

    @vstinner vstinner added the easy label Jun 2, 2017
    @vstinner vstinner changed the title SubinterpThreadingTests.test_threads_join_2() of test_threading leaks references [EASY] SubinterpThreadingTests.test_threads_join_2() of test_threading leaks references Jun 2, 2017
    @sayanchowdhury
    Copy link
    Mannequin

    sayanchowdhury mannequin commented Jun 2, 2017

    I replicated the issue. Starting to work on the issue.

    @vstinner
    Copy link
    Member Author

    vstinner commented Jun 8, 2017

    This issue is a duplicate of bpo-30547.

    I tested that bpo-30547 fixes this issue.

    @vstinner vstinner closed this as completed Jun 8, 2017
    @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 easy performance Performance or resource usage
    Projects
    None yet
    Development

    No branches or pull requests

    1 participant