aho135 commented on PR #20314: URL: https://github.com/apache/druid/pull/20314#issuecomment-5701792181
> Since the underlying Calcite problem has been resolved, now I can't figure out under which cases there will be such problem. Thanks for getting that Calcite patch merged in @FrankChen021 We have some users of the system that are using AI to generate large queries, so for our case the timeout will help protect the system in those situations > I think one more important thing is that we need performance regression test for each PR so that we can know and solve any performance regression before it's merge into the master branch. Yeah agreed, maybe once the Calcite patch is merged in we could at least have a unit test for this -- 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]
