[ARROW-12161] [C++][R] Async streaming CSV reader deadlocking when being run synchronously from datasets

• Type: Bug
• Status: Resolved
• Priority: Blocker
• Resolution: Fixed
• Fix Version/s: 4.0.0
• Component/s: C++, R
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/27982
• Assignee: Weston Pace
• Reporter: Weston Pace
• Created: 31/Mar/21 02:14
• Updated: 11/Jan/23 08:24
• Resolved: 09/Apr/21 21:20

Description

ARROW-11887 added async to the streaming CSV reader.  In order to keep backwards compatibility the old sync API simply calls the async API and waits for it to finish.  However, that wait cannot happen safely in a "nested" context (e.g. dataset reading).

For example, imagine two cores.  The dataset read launches two CSV scans.  Each scan occupies a core waiting for a future.  Those futures are being filled by I/O threads.  The I/O threads finish and go to transfer.  The transfer cannot happen because the CPU executor is filled.

This will be fixed as part of ARROW-7001 but that still some ways away.  An easier change might be to take some of the 7001 changes and include them as part of the 11887 feature.

Issue Links

• links to: GitHub Pull Request #9868
• links to: GitHub Pull Request #10018
• links to: GitHub Pull Request #10019