maytasm commented on pull request #11626:
URL: https://github.com/apache/druid/pull/11626#issuecomment-904484786


   > I don't really understand the reason for this to exist in the core Druid 
expression system, both expressions and SQL operators can be defined in 
extensions if this is needed for some sort of test, and it seems a potential 
risk to allow queries to run basically indefinitely(?) and hold resources.
   
   It's undocumented so I think there is less chance of anyone actually using 
this in production in non-test scenario. I think refactoring this into an 
extension for test purposes codes is also a good idea.


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

Reply via email to