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

In function arrow::compute::MakeExecBatch it checks for the input Datum's type id:

Datum::type() can return null pointers. This will happen when the Datum is constructed from a record batch or a table:

However, the MakeExecBatch function does not check whether the datum is a table before Datum::type() is used. This leads to a null pointer dereference when the id() function is called.

I suggest we additionally check that Datum::type() does not return a null pointer before calling id() on it.

Component(s)

C++