[GitHub] jon-wei commented on issue #6559: Backpressure metric should not be emitted by default

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

jon-wei commented on issue #6559: Backpressure metric should not be emitted by default
URL: https://github.com/apache/incubator-druid/issues/6559#issuecomment-435238082
 
 
   Some thoughts:
   - Setting aside the issue of how to control whether a metric is emitted: for whether a new metric should be emitted by default or not, we should evaluate the new metric case-by-case, I don't think a blanket policy of "no new default metrics" is appropriate. This backpressure metric seems reasonable to emit by default to me.
   - I agree with @gianm, I think it's excessive to require cluster operators to write an extension to control what metrics are emitted, this should be handled with configuration. A cluster operator is not necessarily a developer, and may not even have access to someone who could write such an extension.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

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