benbellick commented on code in PR #26835:
URL: https://github.com/apache/datafusion/pull/26835#discussion_r4413215551
##########
datafusion/expr/src/udf.rs:
##########
@@ -1015,6 +1030,26 @@ pub trait ScalarUDFImpl: Debug + DynEq + DynHash + Send
+ Sync + Any {
false
}
+ /// Returns whether this function has been audited for range-key locality
+ /// analysis with these argument types and literal values.
+ ///
+ /// The safety requirements of
[`Self::supports_range_key_locality_analysis`]
+ /// apply to every admitted combination. Override this method when
admission
+ /// depends on literal values rather than argument types alone.
+ ///
+ /// `literal_arguments` has one entry per argument. `None` means the
argument
+ /// is not a literal; `Some(value)` supplies its value, which may itself be
+ /// NULL. Non-literal expressions are not evaluated to obtain these values.
+ ///
+ /// The default delegates to the type-only hook, preserving existing
opt-ins.
+ fn supports_range_key_locality_analysis_with_args(
Review Comment:
https://github.com/apache/datafusion/commit/d3dbb82014f8c97c075075611287d7c5885cc2c0
--
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]
