Skip to content

Commit e146840

Browse files
committed
Minor: Add comment explaining rationale to avoid dependencies on functions
1 parent d1a3058 commit e146840

File tree

1 file changed

+8
-0
lines changed

1 file changed

+8
-0
lines changed

datafusion/optimizer/Cargo.toml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -43,6 +43,14 @@ name = "datafusion_optimizer"
4343
[features]
4444
recursive_protection = ["dep:recursive"]
4545

46+
# Note -- please DO NOT add a dependency here to any of the datafusion-functions
47+
# crates. While it is tempting to try and add an optimizer pass that uses
48+
# datafusion-functions Doing so makes it harder for downstream crates to
49+
# provide their own function library and smaller install footprint.
50+
#
51+
# If you want to add special handling for a specific function, use the methods
52+
# on the ScalarUDFImpl or AggregateUDFImpl traits (or add a new method to those
53+
# traits).
4654
[dependencies]
4755
arrow = { workspace = true }
4856
chrono = { workspace = true }

0 commit comments

Comments
 (0)