voonhous commented on issue #19638:
URL: https://github.com/apache/hudi/issues/19638#issuecomment-5559550232

   @anshulsingh-py Following up after reviewing #19850: the latest 
implementation uses the evaluator’s own resolver during validation and rejects 
expressions it cannot resolve. This addresses the reported procedure-filter 
behavior without expanding the supported function set. I’ve added more detail 
in [my PR 
reply](https://github.com/apache/hudi/pull/19850#issuecomment-5559548912).
   
   The PR looks good to me, pending CI. Your `FunctionRegistry` proposal 
remains useful as a separate enhancement—please proceed with the follow-up 
issue as discussed. We can also track making direct `evaluateFilter()` calls 
fail explicitly when validation is omitted.
   


-- 
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]

Reply via email to