[GitHub] [pinot] richardstartin commented on pull request #7114: Support REGEXP_EXTRACT

[GitBox <gi...@apache.org>]

richardstartin commented on pull request #7114:
URL: https://github.com/apache/pinot/pull/7114#issuecomment-920119827


   > Actually I think there might be some optimization we can do when LITERAL is passed in by making the FunctionInfo a bit stateful. but that's for a different optimization PR.
   
   Are you thinking about precompiling the pattern? I was thinking the same regarding the JsonPath functions where precompilation would improve performance, I also think it would better to be able to e.g. validate and clean inputs once. 


-- 
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: commits-unsubscribe@pinot.apache.org

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



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org