shauryachats commented on PR #17709: URL: https://github.com/apache/pinot/pull/17709#issuecomment-3923636783
+1. @Akanksha-kedia as I mentioned in your previous PR https://github.com/apache/pinot/pull/17647: > @Akanksha-kedia is this only for queries executed through Controller UI? If yes, could we instead consider modifying only the controller APIs to additionally bundle the controller execution time given that most of the queries would generally be through the broker itself and this increases the response size without providing any additional information for broker queries. Additionally, if this information is only supposed to be bubbled up to the Controller UI, you can instead modify the UI code to expose the actual latency of the controller endpoint. -- 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]
