This is an automated email from the ASF dual-hosted git repository.
comphead pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/datafusion.git
The following commit(s) were added to refs/heads/main by this push:
new 63a10ba080 Add some comments about adding new dependencies in
datafusion-sql (#16543)
63a10ba080 is described below
commit 63a10ba080d6606f627573b9546956702a9f340f
Author: Andrew Lamb <and...@nerdnetworks.org>
AuthorDate: Tue Jun 24 18:19:39 2025 -0400
Add some comments about adding new dependencies in datafusion-sql (#16543)
---
datafusion/sql/Cargo.toml | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/datafusion/sql/Cargo.toml b/datafusion/sql/Cargo.toml
index b778db4676..eca40c5532 100644
--- a/datafusion/sql/Cargo.toml
+++ b/datafusion/sql/Cargo.toml
@@ -43,6 +43,10 @@ unicode_expressions = []
unparser = []
recursive_protection = ["dep:recursive"]
+# Note the sql planner should not depend directly on the datafusion-function
packages
+# so that it can be used in a standalone manner with other function
implementations.
+#
+# They are used for testing purposes only, so they are in the dev-dependencies
section.
[dependencies]
arrow = { workspace = true }
bigdecimal = { workspace = true }
@@ -56,6 +60,7 @@ sqlparser = { workspace = true }
[dev-dependencies]
ctor = { workspace = true }
+# please do not move these dependencies to the main dependencies section
datafusion-functions = { workspace = true, default-features = true }
datafusion-functions-aggregate = { workspace = true }
datafusion-functions-nested = { workspace = true }
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@datafusion.apache.org
For additional commands, e-mail: commits-h...@datafusion.apache.org
