[GitHub] [druid] FrankChen021 commented on pull request #12126: Introduce query/long/count metric and add long query identifier to request log

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

FrankChen021 commented on pull request #12126:
URL: https://github.com/apache/druid/pull/12126#issuecomment-1006540824


   Thanks for your eyes on Druid and the contribution. 
   
   1st, I think the term 'slowQuery' might be a better and widely used name other than 'longQuery'.
   
   2nd, I'm still confused why you want to count the number of longQuery based on threshold defined in Druid? We usually do such things in a 3rd metric visualization system. I'm doubtful if it will be accepted by the community.


-- 
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@druid.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org