[GitHub] [pinot] amrishlal commented on pull request #8159: Adding broker level config for disabling Pinot queries with Groovy

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

amrishlal commented on pull request #8159:
URL: https://github.com/apache/pinot/pull/8159#issuecomment-1032309087


   This will scan each and every statement when DISABLE_GRROVY option is enabled. I am wondering if there is a more efficient option like doing a check while invoking groovy function whether it is disabled for SQL statements or not? I don't think there is an issue with groovy function being called during ingestion so that can be ignored.


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