[C++] Missing joinable check for thread in AsofJoinNode Destructor

[mroz45]

Describe the bug, including details regarding any error messages, version, and platform.

In the destructor of the AsofJoinNode class, no check is performed to ensure that the process_thread_ is joinable before calling the join() method. This can lead to an std::system_error exception if the thread has not been initialized, which can happen if the creation of the plan fails.

Component(s)

C++

[raulcd]

Here the code being mentioned for context:

arrow/cpp/src/arrow/acero/asof_join_node.cc

Lines 1136 to 1141 in 802a846

	virtual ~AsofJoinNode() {
	#ifdef ARROW_ENABLE_THREADING
	PushProcess(false);
	process_thread_.join();
	#endif
	}

CC @zanmato1984 @westonpace

[zanmato1984]

Hi @mroz45 , thanks for reporting this. Are you willing to submit a fix? And I can help to review. It's OK if not, then I'll take a look soon.

[mroz45]

Hi @zanmato1984, I am working on this, and I will add a PR soon.