andygrove commented on issue #630:
URL:
https://github.com/apache/datafusion-comet/issues/630#issuecomment-2221344966
> Hmm, would it make sense to use just DataFusionError for those? I think
for a 3rd party user (like me) it would be fine, and just easier if I only need
to handle one type of error coming from DF + these expressions, but dunno if
Comet internally needs something else from the errors.
Yes, I ended up using DataFusionError but wrapping a Spark-specific error.
For example:
```rust
Err(DataFusionError::External(Box::new(
SparkError::ArithmeticOverflow(self.data_type_name.clone()),
)))
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
