[CHORE] Add error snafus for local executor

[colin-ho]

Example:

df = daft.from_pydict({"a": ["foo", "bar", "baz"]})
df = df.select(df["a"].str.match("["))
df.show()
Error when running pipeline node ProjectOperator
Traceback (most recent call last):
  File "test.py", line 21, in <module>
    df.show()
  File "/Users/colinho/Desktop/Daft/daft/api_annotations.py", line 26, in _wrap
    return timed_method(*args, **kwargs)
  File "/Users/colinho/Desktop/Daft/daft/analytics.py", line 185, in tracked_method
    return method(*args, **kwargs)
  File "/Users/colinho/Desktop/Daft/daft/dataframe/dataframe.py", line 2278, in show
    dataframe_display = self._construct_show_display(n)
  File "/Users/colinho/Desktop/Daft/daft/dataframe/dataframe.py", line 2235, in _construct_show_display
    for table in get_context().runner().run_iter_tables(builder, results_buffer_size=1):
  File "/Users/colinho/Desktop/Daft/daft/runners/pyrunner.py", line 216, in run_iter_tables
    for result in self.run_iter(builder, results_buffer_size=results_buffer_size):
  File "/Users/colinho/Desktop/Daft/daft/runners/pyrunner.py", line 197, in run_iter
    results_gen = executor.run(
  File "/Users/colinho/Desktop/Daft/daft/execution/native_executor.py", line 33, in run
    PyMaterializedResult(MicroPartition._from_pymicropartition(part)) for part in self._executor.run(psets_mp)
daft.exceptions.DaftCoreException: DaftError::ValueError regex parse error:
    [
    ^
error: unclosed character class

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 118 lines in your changes missing coverage. Please review.

Please upload report for BASE (main@b15b0c7). Learn more about missing BASE report.
Report is 1 commits behind head on main.

Files	Patch %	Lines
...c/daft-local-execution/src/sinks/streaming_sink.rs	0.00%	32 Missing ⚠️
...rc/daft-local-execution/src/sinks/blocking_sink.rs	0.00%	24 Missing ⚠️
src/daft-local-execution/src/sinks/hash_join.rs	0.00%	23 Missing ⚠️
src/daft-local-execution/src/lib.rs	0.00%	13 Missing ⚠️
...-execution/src/intermediate_ops/intermediate_op.rs	0.00%	11 Missing ⚠️
src/daft-local-execution/src/sources/in_memory.rs	0.00%	7 Missing ⚠️
src/daft-local-execution/src/pipeline.rs	0.00%	5 Missing ⚠️
src/daft-local-execution/src/run.rs	0.00%	2 Missing ⚠️
src/daft-local-execution/src/sources/source.rs	0.00%	1 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #2660   +/-   ##
=======================================
  Coverage        ?   63.53%           
=======================================
  Files           ?      981           
  Lines           ?   111997           
  Branches        ?        0           
=======================================
  Hits            ?    71152           
  Misses          ?    40845           
  Partials        ?        0           

Files	Coverage Δ
src/daft-local-execution/src/sources/scan_task.rs	0.00% <ø> (ø)
src/daft-local-execution/src/sources/source.rs	0.00% <0.00%> (ø)
src/daft-local-execution/src/run.rs	0.99% <0.00%> (ø)
src/daft-local-execution/src/pipeline.rs	0.00% <0.00%> (ø)
src/daft-local-execution/src/sources/in_memory.rs	0.00% <0.00%> (ø)
...-execution/src/intermediate_ops/intermediate_op.rs	0.00% <0.00%> (ø)
src/daft-local-execution/src/lib.rs	10.52% <0.00%> (ø)
src/daft-local-execution/src/sinks/hash_join.rs	0.00% <0.00%> (ø)
...rc/daft-local-execution/src/sinks/blocking_sink.rs	0.00% <0.00%> (ø)
...c/daft-local-execution/src/sinks/streaming_sink.rs	0.00% <0.00%> (ø)

[samster25]

We probably don't want to nest the execution error under DaftError::External otherwise the conversion of the DaftError to the correct PythonError won't happen.

For now, we can unnest the daft error, log where it came from and then return that.

[colin-ho]

Done and updated PR description example. Assumed you meant not to nest DaftErrors under DaftError::InternalError, the Externals are for the tokio oneshot and join errors.

[samster25]

oh whoops, yeah looks good now!