tustvold commented on code in PR #4999:
URL: https://github.com/apache/arrow-datafusion/pull/4999#discussion_r1082911731


##########
datafusion/physical-expr/Cargo.toml:
##########
@@ -35,12 +35,15 @@ path = "src/lib.rs"
 [features]
 crypto_expressions = ["md-5", "sha2", "blake2", "blake3"]
 default = ["crypto_expressions", "regex_expressions", "unicode_expressions"]

Review Comment:
   > Clearly IOx uses it
   
   That is the point I'm trying to make, IOx **doesn't** use it, at least not 
within any of its tests. A user theoretically could construct a query that 
directly compares dictionary columns, in practice there are extremely limited 
use-cases that come to mind of this.
   
   This feature was only enabled in 
https://github.com/apache/arrow-datafusion/pull/4168 prior to that it was 
disabled



-- 
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...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to