GabrielCWT commented on PR #18686: URL: https://github.com/apache/druid/pull/18686#issuecomment-3454168211
> While an improvement, the impact seems to be only in the micro-second range. Will the effect be more pronounced if the query interval spec has a large number of intervals? @kfaraz, the improvement should be more pronounced when there is a high QPS. The `Interval.toString()` is called when emitting metrics during every query. Obviously with low QPS, this optimisation will be barely noticeable. You are also right that we would see an improvement if there was a large number of intervals. -- 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]
