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.

Sign up for GitHub

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

Jump to bottom

ARROW-12040: [C++] Fix potential deadlock in recursive S3 walks #9842

Closed
wants to merge 4 commits into from
Closed

ARROW-12040: [C++] Fix potential deadlock in recursive S3 walks #9842

wants to merge 4 commits into from

Commits on Apr 6, 2021

  1. ARROW-12040: Switch to using task group instead of manually tracking … 

    …tasks in flight. This makes the walker thread safe so we can get rid of the mutex

ARROW-12040: Slight cleanup
    @westonpace
    westonpace committed Apr 6, 2021
    Configuration menu
    Copy the full SHA
    d8d7bcd View commit details
    Browse the repository at this point in the history

  2. ARROW-12040: The mutex is still necessary to guard the calls to the v… 

    …arious handlers as they are not thread safe
    @westonpace
    westonpace committed Apr 6, 2021
    Configuration menu
    Copy the full SHA
    d81ebc2 View commit details
    Browse the repository at this point in the history

  3. ARROW-12040: Addressing comments from PR

    @westonpace
    westonpace committed Apr 6, 2021
    Configuration menu
    Copy the full SHA
    e362029 View commit details
    Browse the repository at this point in the history

Commits on Apr 7, 2021

  1. Nit: remove unused include

    @pitrou
    pitrou committed Apr 7, 2021
    Configuration menu
    Copy the full SHA
    df50ffe View commit details
    Browse the repository at this point in the history