andygrove commented on issue #630: URL: https://github.com/apache/datafusion-comet/issues/630#issuecomment-2212034526
I created https://github.com/apache/datafusion-comet/pull/637 so that we can start splitting code out into separate crates. I also started looking at what would be involved in creating a `datafusion-comet-spark-expr` crate, and the first issue I ran into is that our `CometError` type has dependencies on the JNI crate, and this doesn't seem ideal for pure Rust downstream projects that just want to re-use the expressions. Perhaps we'll need a separate error type that the new crate can use and that we can convert into a CometError for the Comet/Spark integration. -- 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]
