[ARROW-17936] [R] ExecPlanReader test aborts with a crash

• Type: Improvement
• Status: Closed
• Priority: Blocker
• Resolution: Not A Problem
• Fix Version/s: 10.0.0
• Component/s: R
• External issue URL: https://github.com/apache/arrow/issues/33144
• Assignee: Dewey Dunnington
• Reporter: Dewey Dunnington
• Created: 05/Oct/22 01:22
• Updated: 11/Jan/23 11:57
• Resolved: 19/Oct/22 13:11

Description

This just came up in a CI log for an unrelated PR:

https://github.com/apache/arrow/actions/runs/3184705101/jobs/5193539021#step:7:26299

Start test: ExecPlanReader does not start evaluating a query
  test-query-engine.R:27:3 [success]
R: /arrow/cpp/src/arrow/util/async_generator.h:1736: arrow::BackgroundGenerator<T>::Cleanup::~Cleanup() [with T = std::optional<arrow::compute::ExecBatch>]: Assertion `state->worker_thread_id.load() != ::arrow::internal::GetThreadId()' failed.
Aborted (core dumped)

The ExecPlanReader was added in ARROW-17178...the part that errors involves evaluating R code on the main R thread, an exception getting thrown and caught, and some other safe call into R infrastructure that was updated recently.