timsaucer commented on PR #17332:
URL: https://github.com/apache/datafusion/pull/17332#issuecomment-3259495898
> I was playing with this PR this morning to see if Comet could use it, and
it looks like `datafusion-sql` is brought in with the `nested_expressions`
feature (which Comet uses). I wonder if we can disentangle that dependency.
I did a quick check and with the latest push + comet I no longer have any
`sql` dependency in the cargo lock file! I *did* have to make a change in
comet, which was to set `datafusion-functions-nested = {
path="../../../datafusion/datafusion/functions-nested", default-features =
false }` in `native/core/Cargo.toml`.
--
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: github-unsubscr...@datafusion.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org
